You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/12/15 13:27:26 UTC

[GitHub] [accumulo-testing] ctubbsii commented on a change in pull request #178: fix javadoc warnings

ctubbsii commented on a change in pull request #178:
URL: https://github.com/apache/accumulo-testing/pull/178#discussion_r769610070



##########
File path: src/main/java/org/apache/accumulo/testing/continuous/UndefinedAnalyzer.java
##########
@@ -247,6 +247,11 @@ TabletAssignment findMostRecentAssignment(String row, long time1, long time2) {
   /**
    * Class to analyze undefined references and accumulo logs to isolate the time/tablet where data
    * was lost.
+   * 
+   * @param args
+   *          the command line arguments
+   * @throws Exception
+   *           if an exception occurs

Review comment:
       I think you have your warnings set weird. We normally exclude missing standard javadocs from warnings, because it forces us to put things in like this, which doesn't add any value from what you can see already from the method signature.




-- 
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: notifications-unsubscribe@accumulo.apache.org

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