qzyu999 commented on PR #50232: URL: https://github.com/apache/arrow/pull/50232#issuecomment-4815344801
Force-pushed the shredding implementation built on top of the refactored decoding/encoding layers. This uses the idiomatic C++ view classes and RAII builder from the parent PRs. Highlights: - Template-refactored shredding loops (`ShredPrimitiveLoop<>`, `ShredBinaryLoop<>`) eliminate per-type code duplication - Recursive array element shredding (Rust parity) - All 5 list-like types supported in reconstruction (List, LargeList, FixedSizeList, ListView, LargeListView) - `out_null_bitmap` parameter for SQL NULL disambiguation (Rust NullBuffer parity) - Metadata caching in reconstruction path (avoids redundant `DecodeMetadata` per row) - Object sub-field native extraction (primitives only; Object/Array sub-schemas deferred) 335 tests pass with `BUILD_WARNING_LEVEL=CHECKIN` (114 shredding-specific). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
