chia7712 commented on code in PR #17417:
URL: https://github.com/apache/kafka/pull/17417#discussion_r1792702186
##########
coordinator-common/src/main/java/org/apache/kafka/coordinator/common/runtime/MultiThreadedEventProcessor.java:
##########
@@ -90,6 +90,7 @@ public MultiThreadedEventProcessor(
* @param time The time.
* @param eventAccumulator The event accumulator.
*/
+ @SuppressWarnings("this-escape")
public MultiThreadedEventProcessor(
Review Comment:
This class is not used in mocking, so could you please add `final` to the
class instead of suppress?
see #17409 for example
--
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]
