jsancio commented on a change in pull request #10883:
URL: https://github.com/apache/kafka/pull/10883#discussion_r651929028
##########
File path: core/src/main/scala/kafka/tools/TestRaftServer.scala
##########
@@ -238,6 +238,8 @@ class TestRaftServer(
reader.close()
case Shutdown => // Ignore shutdown command
+
+ case _ => // Ignore other events (such as null)
Review comment:
I assume that this value is `null` when the timeout is hit. If that is
the case then make the case `case null => // Ignore null when timeout expires`.
This should both handle the case when it is null and give us an error if all
of the cases/type are not handled.
--
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
