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

Geo spatial: Geography #2830

Closed
wants to merge 8 commits into from
Closed

Conversation

jievince
Copy link
Contributor

@jievince jievince commented Sep 9, 2021

This PR is deprecated!!!
This PR is divided to those PRs:

close #2595

Some extra notes:

  • Our macro MUST_USE_RESULT conflicts with s2's third-party's macro, so I change it to NG_MUST_USE_RESULT

@jievince jievince added the wip Solution: work in progress label Sep 9, 2021
@jievince jievince force-pushed the geo-spatial branch 3 times, most recently from d770167 to 7bc1e89 Compare September 15, 2021 09:03
@jievince jievince force-pushed the geo-spatial branch 10 times, most recently from 35bd55a to 5a66a88 Compare September 22, 2021 03:24
@Sophie-Xie Sophie-Xie added this to the v2.6.0 milestone Sep 22, 2021
@yixinglu
Copy link
Contributor

Don't forget to update TCK parser to parse the wkt data format

@jievince jievince force-pushed the geo-spatial branch 2 times, most recently from c104dbd to 7594f2c Compare September 23, 2021 08:42
@jievince jievince requested a review from czpmango September 26, 2021 03:13
@jievince jievince closed this Oct 12, 2021
@jievince jievince deleted the geo-spatial branch October 25, 2021 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wip Solution: work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[QE]Geo-Code support (Geo data type, index, etc.?)
3 participants