wenjin272 commented on issue #872: URL: https://github.com/apache/flink-agents/issues/872#issuecomment-4891486403
Thanks to @weiqingy and @zxs1633079383 for the discussion. Below are my thoughts. 1. I agree that, without injecting TypeInformation, the two approaches are effectively equivalent, and we can simply use Kryo directly. 2. I believe we must ensure state compatibility at least within a major version, which means we should pin the Kryo version for the duration of that major release. 3. I think a single JSON document is enough. 4. I prefer folding everything into one binary change, because I believe these PRs will all be merged within 0.4. -- 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]
