apurtell commented on a change in pull request #1879:
URL: https://github.com/apache/hbase/pull/1879#discussion_r437737505
##########
File path:
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/HQuorumPeer.java
##########
@@ -85,14 +85,20 @@ public static void main(String[] args) {
}
private static void runZKServer(QuorumPeerConfig zkConfig) throws
UnknownHostException, IOException {
Review comment:
The current signature proposes we throw IOException or UnknownHostException.
ZK 3.6 throws another type of checked exception, which causes a compilation
error. Therefore I catch that and potentially others, and wrap it into an IOE.
Adding the new checked exception to the signature would cause a compilation
problem with 3.4, so that is not possible due to this legacy.
----------------------------------------------------------------
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]
