[
https://issues.apache.org/jira/browse/PARQUET-1303?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16484062#comment-16484062
]
ASF GitHub Bot commented on PARQUET-1303:
-----------------------------------------
zebehringer commented on a change in pull request #482: PARQUET-1303 correct
ClassCastException for Avro @Stringable fields
URL: https://github.com/apache/parquet-mr/pull/482#discussion_r189926607
##########
File path:
parquet-avro/src/test/java/org/apache/parquet/avro/TestReflectLogicalTypes.java
##########
@@ -426,33 +426,6 @@ public void testWriteNullableUUID() throws IOException {
read(REFLECT, nullableUuidStringSchema, test));
}
- @Test(expected = ClassCastException.class)
Review comment:
I have restored the test, removed the expected exception and added a couple
asserts, similar to testWriteUUID, which uses REFLECT for the write() instead
of uuidSchema. Let me know if that covers it.
----------------------------------------------------------------
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:
[email protected]
> 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
(v7.6.3#76005)
