Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(rs|constructors,transformations): adjust SRID and use native ST_GEOMFROMGEOJSON to return geometries instead of VARCHAR #463

Merged
merged 2 commits into from
Dec 29, 2023

Conversation

vdelacruzb
Copy link
Contributor

@vdelacruzb vdelacruzb commented Dec 29, 2023

Description

Shortcut

Type of change

  • Fix

Acceptance


SELECT vdelacruz_carto.ST_BEZIERSPLINE(
  ST_GEOMFROMTEXT(
    'LINESTRING (-76.091308 18.427501,-76.695556 18.729501,-76.552734 19.40443,-74.61914 19.134789,-73.652343 20.07657,-73.157958 20.210656)'
  ),
  10000,
  0.9
);
-- returns geom

SELECT vdelacruz_carto.ST_MAKEELLIPSE(ST_Point(-73.9385,40.6643), 5, 3, -30, 'miles', 80);
-- returns geom

SELECT ST_SRID(vdelacruz_carto.ST_MAKEENVELOPE(0, 0, 1, 1));
-- 4326

SELECT ST_SRID(vdelacruz_carto.ST_CENTERMEAN(ST_GEOMFROMTEXT('POLYGON ((30 10, 40 40, 20 40, 10 20, 30 10))')));
-- 4326

SELECT ST_SRID(vdelacruz_carto.ST_CENTERMEDIAN(ST_GEOMFROMTEXT('POLYGON ((30 10, 40 40, 20 40, 10 20, 30 10))')));
-- 4326

SELECT ST_SRID(vdelacruz_carto.ST_CENTROID(ST_GEOMFROMTEXT('POLYGON ((30 10, 40 40, 20 40, 10 20, 30 10))')));
-- 4326

SELECT ST_SRID(vdelacruz_carto.ST_DESTINATION(ST_MakePoint(-3.70325,40.4167), 10, 45, 'miles'));
-- 4326

SELECT ST_SRID(vdelacruz_carto.ST_GREATCIRCLE(ST_MAKEPOINT(-3.70325,40.4167), ST_MAKEPOINT(-73.9385,40.6643), 20));
-- 4326

Copy link

This pull request has been linked to Shortcut Story #194555: Fix Redshift ST_GEOMFROM in AT.

@vdelacruzb vdelacruzb requested a review from jgoizueta December 29, 2023 13:02
@vdelacruzb vdelacruzb merged commit 7b9961f into main Dec 29, 2023
8 checks passed
@vdelacruzb vdelacruzb deleted the bug/sc-194555/fix-redshift-st-geomfrom-in-at branch December 29, 2023 14:46
@vdelacruzb vdelacruzb mentioned this pull request Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants