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


   All deprecation warnings *will go away*. Style and javadocs will be corrected
   
   ```
   
./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/impl/IOStatisticsBinding.java:450:
      DurationTracker tracker = createTracker(factory, statistic);: 'method 
def' child has incorrect indentation level 6, expected level should be 4. 
[Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/impl/IOStatisticsBinding.java:451:
      try {: 'try' has incorrect indentation level 6, expected level should be 
4. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/impl/IOStatisticsBinding.java:453:
        input.apply();: 'try' child has incorrect indentation level 8, expected 
level should be 6. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/impl/IOStatisticsBinding.java:454:
      } catch (IOException | RuntimeException e) {: 'try rcurly' has incorrect 
indentation level 6, expected level should be 4. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/impl/IOStatisticsBinding.java:456:
        tracker.failed();: 'catch' child has incorrect indentation level 8, 
expected level should be 6. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/impl/IOStatisticsBinding.java:458:
        throw e;: 'catch' child has incorrect indentation level 8, expected 
level should be 6. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/impl/IOStatisticsBinding.java:459:
      } finally {: 'catch rcurly' has incorrect indentation level 6, expected 
level should be 4. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/impl/IOStatisticsBinding.java:463:
        tracker.close();: 'finally' child has incorrect indentation level 8, 
expected level should be 6. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/statistics/impl/IOStatisticsBinding.java:464:
      }: 'finally rcurly' has incorrect indentation level 6, expected level 
should be 4. [Indentation]
   
./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/test/java/org/apache/hadoop/fs/contract/AbstractContractStreamIOStatisticsTest.java:77:
  /**: First sentence should end with a period. [JavadocStyle]
   ```


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to