Gabriel39 opened a new pull request, #67921:
URL: https://github.com/apache/doris/pull/67921

   ### What problem does this PR solve?
   
   Follow-up to #67325; partially addresses #66496.
   
   Lance columns containing Arrow Null inside lists, structs, or map values are 
currently reported as unsupported. This change maps Null leaves recursively, 
materializes them without accessing nonexistent Arrow validity/value buffers, 
and preserves their logical `NULL_TYPE` through nested schema RPC serialization.
   
   Array offsets, parent nullability, empty collections, and sliced inputs 
remain intact. Unknown extensions continue to be rejected.
   
   ### Tests
   
   - FE type-conversion tests: 10 passed in an isolated runner using existing 
dependencies; the new nested mapping test failed before the change.
   - BE nested Null tests: 9 passed in an isolated harness using the changed 
routines and compatible cached dependencies. The original schema conversion, 
RPC type preservation, and nested read failures were reproduced.
   - All affected C++ translation units passed syntax checks against the 
current source headers and generated schemas.
   - FE `mvn checkstyle:check`, clang-format 16 checks, and `git diff --check` 
passed.
   - Added S3 TVF regression coverage and a reproducible seven-column Lance 
fixture, verified by PyLance round-trip reading.
   
   Full FE Maven tests are blocked locally by a Thrift compiler/runtime version 
mismatch (installed compiler 0.16 versus branch dependency 0.24). The complete 
BE suite and S3 regression have not been run locally and require CI.
   
   ### Release note
   
   Support Arrow Null elements and fields nested in Lance arrays, structs, and 
map values.
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to