bodduv opened a new issue, #1109:
URL: https://github.com/apache/arrow-java/issues/1109

   ### Describe the bug, including details regarding any error messages, 
version, and platform.
   
   https://github.com/apache/arrow-java/pull/892 made efforts to streamline and 
simplify extension readers and writers. In this, `ComplexCopier.copy` relies on 
`FieldReader.getField` being defined for extension types.
   
   Usages of `ComplexCopier.copy` with readers backed by vector, e.g., 
`UuidReaderImpl` and `VariantReaderImpl` that do have `getField` 
implementation, should be OK (as there is concrete notion of Field), but if 
holder readers, namely, `NullableUuidHolderReaderImpl` or 
`NullableVariantHolderReaderImpl` are used this results in an NPE.
   
   Having said that `getField` method for holder readers appears to be a 
misnomer as we do not have a notion of a field for the holders. This can be 
seen from the absence of `getField` overrides for primitive types.


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

Reply via email to