Skip to content

ST_Centroid: Centroid not Envelope Center#153

Merged
randallwhitman merged 3 commits intomasterfrom
centroid
May 18, 2018
Merged

ST_Centroid: Centroid not Envelope Center#153
randallwhitman merged 3 commits intomasterfrom
centroid

Conversation

@randallwhitman
Copy link
Copy Markdown
Contributor

@randallwhitman randallwhitman commented May 17, 2018

Centroid method is recently available in Geometry API.
fyi @stolstov @mbasmanova
#150

@randallwhitman randallwhitman requested a review from climbage May 17, 2018 20:32
select ST_AsJson(ST_GeometryN(ST_GeomFromText('multilinestring ((2 4, 10 10), (20 20, 7 8))'), 2)) from onerow;
select ST_AsText(ST_GeometryN(ST_GeomFromText('multipolygon (((3 3, 4 6, 5 3, 3 3)),((8 24, 1 28, 9 25, 8 24)), ((13 33, 7 36, 1 40, 10 43, 13 33)))'), 1)) from onerow;
select ST_AsText(ST_InteriorRingN(ST_Polygon('polygon ((0 0, 8 0, 0 8, 0 0), (1 1, 1 5, 5 1, 1 1))'), 1)) from onerow;
select ST_AsJson(ST_Polygon(1.5,2.5, 3.0,2.2, 2.2,1.1));
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

from <table> is no longer necessary as of Hive-0.12

@randallwhitman
Copy link
Copy Markdown
Contributor Author

After updating the hiverc file to point to the updated versions of both Geometry and Spatial Framework, I was able to test ST-Centroid on both Hive and SparkSQL.

@randallwhitman randallwhitman merged commit 93cb31c into master May 18, 2018
@randallwhitman randallwhitman added this to the v2.1 milestone Jun 29, 2018
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.

1 participant