ORC-1920: [C++] Support `Geometry` and `Geography` types
What changes were proposed in this pull request? Support Geometry and Geography types for c++ side
Why are the changes needed? Add support for Geometry and Geography types
How was this patch tested? UT passed
Was this patch authored or co-authored using generative AI tooling? No
Gentle ping, @ffacs .
Gentle ping, @ffacs .
Pong. Thank you for your review @dongjoon-hyun , I'd update this patch these days.
Thank you, @ffacs . Also, please participate the 1.8.10 vote too when you have some time.
Could you fix the above comments, @ffacs ?
Let me know when ready to review.
@wgtmac @dongjoon-hyun I'm ready to review now, please take a look when you're free~
Thank you, @ffacs .
Is this ready or do we need more revision, @ffacs and @wgtmac ?
Is this ready or do we need more revision, @ffacs and @wgtmac ?
My only concern is the name of public api: https://github.com/apache/orc/pull/2269/files#r2184135226. It is better to use getCrs() and getAlgorithm()/getEdgeAlgorithm()
Thank you everyone, @ffacs , @williamhyun , @wgtmac . Merged to main/2.2.