GH-45522: [Parquet][C++] Parquet GEOMETRY and GEOGRAPHY logical type implementations#45459

Open

paleolimbot wants to merge 193 commits intoapache:mainfrom paleolimbot:Kontinuation-kontinuation-parquet-geometry

+7,504-2,406

Commits on Feb 6, 2025

update thrift
paleolimbot
committed
update thrift defs
paleolimbot
committed
add stubs
paleolimbot
committed
split methods out of line
paleolimbot
committed
maybe to/from thrift
paleolimbot
committed
a few more serializers
paleolimbot
committed
add basic test for serialization
paleolimbot
committed
add sort order check
paleolimbot
committed
update thrift
paleolimbot
committed
start geom utiles
paleolimbot
committed
test roundtrip thrift cases
paleolimbot
committed
more geometry utils
paleolimbot
committed
more
paleolimbot
committed
bounder
paleolimbot
committed
add basic test
paleolimbot
committed
a few more strings
paleolimbot
committed
test some bounding box things
paleolimbot
committed
more tests
paleolimbot
committed
fix test
paleolimbot
committed
with passing tests
paleolimbot
committed
add in WKT equiv
paleolimbot
committed
more tests
paleolimbot
committed
start on stats
paleolimbot
committed
implement update/merge for geometry statistics
paleolimbot
committed
more complete stats
paleolimbot
committed
start on factory methods
paleolimbot
committed
more stats things
paleolimbot
committed
maybe work with serde
paleolimbot
committed
Update cpp/src/parquet/types.cc

paleolimbot
and
wgtmac
committed
Updated parquet.thrift and re-generated cpp sources

Kontinuation
authored and
paleolimbot
committed
Geometry value writer could make use of the geometry statistics class to

Kontinuation
authored and
paleolimbot
committed
Geometry column writer now populates correct statistics

Kontinuation
authored and
paleolimbot
committed
format/tidy

Kontinuation
authored and
paleolimbot
committed
Run clang-tidy

Kontinuation
authored and
paleolimbot
committed
Added a test that writes and reads a parquet file containing a geometry column

Kontinuation
authored and
paleolimbot
committed
Remove redundant include

Kontinuation
authored and
paleolimbot
committed
Fix problems found by reviewers

Kontinuation
authored and
paleolimbot
committed
Try to make it build properly on other platforms

Kontinuation
authored and
paleolimbot
committed
Address review comments in apache#43196

Kontinuation
authored and
paleolimbot
committed
Resolve compile errors for MSVC

Kontinuation
authored and
paleolimbot
committed
Expose getters in GeometryStatistics, Change geometry_types from

Kontinuation
authored and
paleolimbot
committed
Add test case for UpdateSpaced, don't generate min/max stats for geometry

Kontinuation
authored and
paleolimbot
committed
Support covering

Kontinuation
authored and
paleolimbot
committed
MakeStatistics and Statistics::Make should not be a breaking change

Kontinuation
authored and
paleolimbot
committed
ColumnIndex, as well as some other fixes and refacturings

Kontinuation
authored and
paleolimbot
committed
Fix compiler warnings on AMD platforms as well as sanitizer warnings

Kontinuation
authored and
paleolimbot
committed
Remove all newly added include directives

Kontinuation
authored and
paleolimbot
committed
include cmath for std::isnan

Kontinuation
authored and
paleolimbot
committed
Test writing WKB encoded geometries using WriteArrow

Kontinuation
authored and
paleolimbot
committed
Change the sort order of geometry from unknown to unsigned; resolved several review comments

Kontinuation
authored and
paleolimbot
committed
Add generate_covering_ member to be explicit that' we'll generate the coverings from bounding box when populating the encoded statistics

Kontinuation
authored and
paleolimbot
committed
Refactor unscoped enums in geometry_util_internal to enum classes

Kontinuation
authored and
paleolimbot
committed
Revert more special case handling for unknown sort order

Kontinuation
authored and
paleolimbot
committed
Fix WKB covering test to take native endianness into consideration

Kontinuation
authored and
paleolimbot
committed
min/max of geometry columns are the WKB representation of lower-left and upper-right points

Kontinuation
authored and
paleolimbot
committed
Address latest review comments

Kontinuation
authored and
paleolimbot
committed
A better implementation of geometry min/max statistics

Kontinuation
authored and
paleolimbot
committed
Update the code to accomodate the latest changes of the standard:

Kontinuation
authored and
paleolimbot
committed
Fix problem decoding WKB geometries with more than 32 coordinates

Kontinuation
authored and
paleolimbot
committed
Re-implemented geometry statistics according to the updated spec:

Kontinuation
authored and
paleolimbot
committed
Revert some unnecessary changes

Kontinuation
authored and
paleolimbot
committed
update so that it all builds
paleolimbot
committed
tests passing!
paleolimbot
committed

Commits on Feb 7, 2025

Commits on Feb 11, 2025

Commits on Feb 14, 2025

Commits on Feb 21, 2025

Commits on Feb 26, 2025

Commits on Feb 27, 2025