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

Posted to issues@flink.apache.org by "dawidwys (via GitHub)" <gi...@apache.org> on 2023/03/06 10:56:57 UTC

[GitHub] [flink] dawidwys commented on pull request #21636: [FLINK-30113] Implement state compression for broadcast and regular operator states

dawidwys commented on PR #21636:
URL: https://github.com/apache/flink/pull/21636#issuecomment-1455911732

   What you did for versioning looks sane. I think the problem is with line https://github.com/apache/flink/blob/a4a11a389609deb5d1de139115cedc35b675b366/flink-runtime/src/main/java/org/apache/flink/runtime/state/metainfo/StateMetaInfoSnapshotReadersWriters.java#L68 which has not been implemented in a future proof way. The intent there was to throw an exception for `readVersion < 6`, not less than `CURRENT_STATE_META_INFO_SNAPSHOT_VERSION`. If we change the condition to `readVersion < 6` it should 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.

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

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