rustyconover commented on PR #966: URL: https://github.com/apache/arrow-nanoarrow/pull/966#issuecomment-6219421488
Thanks! Agreed on the gotcha with lower validation levels. I ran into exactly that: decoding at `NANOARROW_VALIDATION_LEVEL_MINIMAL` (to match what Arrow C++'s reader checks) skips `ArrowArrayFinalizeBuffers()`, so the empty offsets buffer is exported unchanged. I worked around it on my side. It might be worth decoupling the zero offset (and the existing non-NULL data pointer) from the validation level for arrays we know are on the CPU, which the IPC decoder's output always is. I'm happy to do that as a follow-up if you think it's worthwhile. -- 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]
