Hello Balazs Hevele, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24552
to look at the new patch set (#6).
Change subject: POC IMPALA-15165: Add minimal C++ WKB serialization support
......................................................................
POC IMPALA-15165: Add minimal C++ WKB serialization support
Introduce the C++ foundation for reading and writing OGC Well-Known
Binary geometries, independent of the native ST_ function
implementations that follow. This layer is what the Parquet geometry
bounding-box pushdown depends on.
Adds:
- common.h: shared boost::geometry types (point2d, box2d, ...)
- wkb-format.h: WKB header parsing (ParseWkbHeader) and inline
byte-order read/write helpers
- wkb-serialization.{h,cc}: WKB <-> geometry serialization and
bounding-box computation (ComputeWkbBBox)
- wkb-serialization-test.cc: malformed-input coverage and a
write/read round-trip test per geometry type
- Build wiring: wkb-serialization.cc joins ExprsGeoIr; a new
ExprsGeoTests library is registered for the unified BE tests.
Change-Id: I4d5b2ac5d584e3de7712d8471acbc07d743a4de2
Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
---
M be/CMakeLists.txt
M be/src/exprs/geo/CMakeLists.txt
M be/src/exprs/geo/common.h
A be/src/exprs/geo/wkb-format.h
A be/src/exprs/geo/wkb-serialization-test.cc
A be/src/exprs/geo/wkb-serialization.cc
A be/src/exprs/geo/wkb-serialization.h
7 files changed, 958 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/52/24552/6
--
To view, visit http://gerrit.cloudera.org:8080/24552
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4d5b2ac5d584e3de7712d8471acbc07d743a4de2
Gerrit-Change-Number: 24552
Gerrit-PatchSet: 6
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Balazs Hevele <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
