guoweiM commented on pull request #12132:
URL: https://github.com/apache/flink/pull/12132#issuecomment-629650094
Hi, @aljoscha thanks for your suggestion.
1. yes you are right I should delete the v1 code in the test.
1. The reason I used Operator to generate snapshots at the time is that we
could use a set of code to generate both v1 and v2 snapshots of `BucketState`.
1. Otherwise, you need to write twice for how to generate
`BucketStatev1` and `BucketStateV2`.
1. There is a special process also need to write twice. The path in the
snapshot cannot have an absolute path and needs to be specially processed when
the snapshot is generated.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
