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: [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]
