Describe the bug, including details regarding any error messages, version, and platform.
#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). However, 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.
Describe the bug, including details regarding any error messages, version, and platform.
#892 made efforts to streamline and simplify extension readers and writers. In this,
ComplexCopier.copyrelies onFieldReader.getFieldbeing defined for extension types.Usages of
ComplexCopier.copywith readers backed by vector, e.g.,UuidReaderImplandVariantReaderImplthat do havegetFieldimplementation, should be OK (as there is concrete notion of Field). However, if holder readers, namely,NullableUuidHolderReaderImplorNullableVariantHolderReaderImplare used this results in an NPE.Having said that
getFieldmethod 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 ofgetFieldoverrides for primitive types.