HorizonNet commented on a change in pull request #322: HBASE-22586 Javadoc 
Warnings related to @param tag
URL: https://github.com/apache/hbase/pull/322#discussion_r298756676
 
 

 ##########
 File path: 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestActiveMasterManager.java
 ##########
 @@ -196,8 +196,8 @@ public void testActiveMasterManagerFromZK() throws 
Exception {
 
   /**
    * Assert there is an active master and that it has the specified address.
-   * @param zk
-   * @param thisMasterAddress
+   * @param zk single zookeeper watcher
+   * @param expectedAddress expected master address
    * @throws KeeperException
 
 Review comment:
   Could you please also document these two exceptions when you're already on 
it?

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to