mxtymoshyk opened a new pull request, #40444: URL: https://github.com/apache/beam/pull/40444
Follow-up to #40008. @kennknowles flagged a set of classes there that can be final, so this PR makes them final. It makes 31 classes final and removes their CT_CONSTRUCTOR_THROW suppressions. None of them has a subclass or a Mockito mock. BoundedTrieData keeps its separate IS2_INCONSISTENT_SYNC suppression. StatefulDoFnRunner stays non-final because ExecutableStageDoFnOperator creates an anonymous subclass of it. Main and test code compile, and spotbugsMain passes on the touched modules, including Flink 1.20 and 2.2. I did not run the unit tests locally. This builds on #40008, so the diff includes its commits until that merges. -- 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]
