weiqingy commented on PR #28880: URL: https://github.com/apache/flink/pull/28880#issuecomment-5289350357
@Zakelly @gyfora would either of you have bandwidth to take a look? Thanks! This is the first PR of the FLIP-527 implementation, which you both reviewed and voted on. The change is deliberately small to keep review easy: one `default` method on `TypeSerializerSnapshot` that returns its argument unchanged, plus a test asserting the default is identity. There is no caller in this PR. PR-2 adds the caller and PR-3 the first override, so this one is behavior-neutral on its own. Each PR in the stack depends on the one in front of it, so this is currently what gates the rest. Let me know if you would prefer the work split a different way. -- 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]
