openinx commented on issue #1698:
URL: https://github.com/apache/iceberg/issues/1698#issuecomment-756030511


   Thanks @stevenzwu for bringing this up.   It's indeed an problem for flink 
streaming reader because it depends on the java serialization in 
[StreamingReaderOperator](https://github.com/apache/iceberg/pull/1793/files#diff-22513db68a66d835391d1dad5b0b95ed776dbcc3df7c8df7df5450930f367893R67)
 now,  it's easy to crash when we upgrade iceberg lib version (which changes 
the CombinedScanTask classes) and restart the flink job. 
   
   I'd prefer to define the avro schema in `BaseCombinedScanTask` which is 
similar to `BaseDataFile`, then we could maintain the binary bytes which is 
serialized by avro approach to flink state backend.  Let me evaluate the work. 


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

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to