You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2022/01/07 08:07:00 UTC

[GitHub] [iceberg] kbendick edited a comment on pull request #3857: Flink 1.13: Add SerializableTable test cases.

kbendick edited a comment on pull request #3857:
URL: https://github.com/apache/iceberg/pull/3857#issuecomment-1007212455


   Thanks @openinx 
   
   Some questions for more context:
   
   Is this due to Flink 1.13 only (e.g. fixed in 1.14) or did you just choose to implement in 1.13 for another reason?
   
   Personal thoughts / comments:
   
   The SerializableSupplier is written as is to avoid calls to get on the user-side, but if that's prohibiting Kryo usage then I agree that it should be changed if need be.
   
   We should likely consider updating the usage of SerializableSupplier to be consistent across versions eventually, even if it only affects one version, for ease of backporting etc.


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org