guoweiM commented on pull request #14120: URL: https://github.com/apache/flink/pull/14120#issuecomment-730115130
Thanks @SteNicholas for opening this pr. In general the pr is good to me. I just have some minor comments. 1. Provide more precise filtering. Currently it is fine that we use the “typeutils” to filter because there are already some cases. (For example PrimitiveArraySerializerUpgradeTest.java and BasicTypeSerializerUpgradeTest.java). But I think the filter scope might be a little big. If somebody adds a serializer in the “typeutils” package and does not follow our rule we could not find it. 2. Make the filter consistent in the ‘TypeInfoTestCoverageTest’ and ‘TypeSerializerTestCoverageTest’. I think we might use the same condition to filter the “test/itcase/innercalss”. 3. Maybe we could split the commit to two. One is to add a test for StreamElement. One is to add the `‘TypeSerializerTestCoverageTest’` ---------------------------------------------------------------- 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: [email protected]
