nicknezis commented on PR #25896:
URL: https://github.com/apache/flink/pull/25896#issuecomment-2661216618

   > @kurtostfeld I see your motivation for doing this, but you mention that 
there will not be binary compatibility with v1 state. Do you have an idea on 
how v1 Kyro state would migrate to v2 state after this fix? We are trying to 
encourage v2 migration - and this could be a big inhibitor.
   
   @davidradl Sharing this info in case it helps answer your question. 
@kurtostfeld can correct me if I mistate anything. The original Pull Request, 
which was documented in this 
[FLIP-317](https://cwiki.apache.org/confluence/display/FLINK/FLIP-317%3A+Upgrade+Kryo+from+2.24.0+to+5.5.0)
 provided code to migrate older state forward. But there was some email 
discussion about keeping the Kryo upgrade for Flink 2.x. Because of this, there 
was another merge request created that was simpler and did not include the 
backwards compatibility. I think either Pull Request could work, but we just 
need someone to decide and approve before things get too stale. This [Jira 
ticket](https://issues.apache.org/jira/browse/FLINK-3154) has more discussion 
in the comments. Also this [email 
discussion](https://lists.apache.org/thread/vyov944och394lql76zb3myf2g140fdw).


-- 
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-unsubscr...@flink.apache.org

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

Reply via email to