yittg commented on code in PR #5109:
URL: https://github.com/apache/iceberg/pull/5109#discussion_r904514843
##########
flink/v1.15/flink/src/main/java/org/apache/iceberg/flink/data/FlinkParquetReaders.java:
##########
@@ -482,8 +484,10 @@ protected void addElement(ReusableArrayData reused, E
element) {
@Override
protected ArrayData buildList(ReusableArrayData list) {
- list.setNumElements(writePos);
- return list;
+ // Since ReusableArrayData is not accepted by Flink, use
GenericArrayData temporarily to walk around it.
Review Comment:
Thanks @stevenzwu,
I finally got the point, the ArrayDataSerializer in Flink should be renew
each time because it reuse the BinaryArrayData internally. 'Think we can
change the signature of RowDataUtil#clone to accept a supplier of serializer to
walk around it for now.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
