ASF GitHub Bot commented on PARQUET-1303:

zebehringer commented on issue #482: PARQUET-1303 correct ClassCastException 
for Avro @Stringable fields
URL: https://github.com/apache/parquet-mr/pull/482#issuecomment-389966912
   TestReflectLogicalType.testWriteUUIDMissingLogicalType no longer throws 
ClassCastException after this change. In one sense the test is no longer valid, 
but in a strict sense it is since the UUID field in that test is not annotated 
with @Stringable, but there doesn't seem to be any way to detect that in 
AvroWriteSupport, so it now passing is a side effect of the change.

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

> Avro reflect @Stringable field write error if field not instanceof 
> CharSequence
> -------------------------------------------------------------------------------
>                 Key: PARQUET-1303
>                 URL: https://issues.apache.org/jira/browse/PARQUET-1303
>             Project: Parquet
>          Issue Type: Bug
>          Components: parquet-avro
>    Affects Versions: 1.9.0, 1.10.0
>            Reporter: Zack Behringer
>            Priority: Minor
> Annotate a field in a pojo with org.apache.avro.reflect.Stringable and the 
> schema will consider it to be a String field. AvroWriteSupport.fromAvroString 
> assumes the field is either a Utf8 or CharSequence and does not attempt to 
> use the field class' toString method if it is not.

This message was sent by Atlassian JIRA

Reply via email to