hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [hadoop] steveloughran commented on pull request #2323: HADOOP-16830. Add public IOStatistics API.
Date Fri, 02 Oct 2020 14:47:06 GMT

steveloughran commented on pull request #2323:
URL: https://github.com/apache/hadoop/pull/2323#issuecomment-702777116


   Yetus: javac is all about the deprecation changes fixed in #2324 
   
   checkstyles from new tests & code
   ```
   ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/FunctionsRaisingIOE.java:41:
  * @deprecated use {@link org.apache.hadoop.util.functional.FunctionRaisingIOE}: Line is
longer than 80 characters (found 81). [LineLength]
   ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/FunctionsRaisingIOE.java:54:
  * @deprecated use {@link org.apache.hadoop.util.functional.BiFunctionRaisingIOE}: Line is
longer than 80 characters (found 83). [LineLength]
   ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/FunctionsRaisingIOE.java:65:
  * @deprecated use {@link org.apache.hadoop.util.functional.CallableRaisingIOE}: Line is
longer than 80 characters (found 81). [LineLength]
   ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/BufferedIOStatisticsInputStream.java:51:
 /**: First sentence should end with a period. [JavadocStyle]
   ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/package-info.java:28:
* {@link org.apache.hadoop.fs.statistics.IOStatisticsSource#getIOStatistics()} .: Line is
longer than 80 characters (found 81). [LineLength]
   ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/functional/RemoteIterators.java:412:
   /**: First sentence should end with a period. [JavadocStyle]
   ./hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/statistics/TestDurationTracking.java:112:
       (Integer x) -> x);: 'lambda arguments' has incorrect indentation level 8, expected
level should be one of the following: 10, 12. [Indentation]
   ./hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/statistics/TestDurationTracking.java:200:
 /**: First sentence should end with a period. [JavadocStyle]
   ```
   will fix those


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org


Mime
View raw message