socutes commented on a change in pull request #10749:
URL: https://github.com/apache/kafka/pull/10749#discussion_r638535599



##########
File path: raft/src/main/java/org/apache/kafka/raft/QuorumState.java
##########
@@ -116,7 +117,7 @@ public void initialize(OffsetAndEpoch logEndOffsetAndEpoch) 
throws IOException,
             if (election == null) {
                 election = ElectionState.withUnknownLeader(0, voters);
             }
-        } catch (final IOException e) {
+        } catch (final Exception e) {

Review comment:
       Previously, the IOException would have been thrown here, so you need to 
catch it here. After the KIP handles the IOException, the underlying 
IOException will not be thrown up, but will become other exceptions, such as 
UnCheckeDioException, so you need to handle it this 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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to