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


   ```
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/OpenFileOptions.java:31:@InterfaceAudience.Public:1:
 Utility classes should not have a public or default constructor. 
[HideUtilityClassConstructor]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/OpenFileParameters.java:22:import
 java.util.stream.Collectors;:8: Unused import - java.util.stream.Collectors. 
[UnusedImports]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/OpenFileParameters.java:23:import
 java.util.stream.Stream;:8: Unused import - java.util.stream.Stream. 
[UnusedImports]
   
./hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/impl/S3AOpenFileOperation.java:52:
  public class S3AOpenFileOperation extends AbstractStoreOperation {: 'class 
def modifier' has incorrect indentation level 2, expected level should be 0. 
[Indentation]
   
./hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/impl/S3AOpenFileOperation.java:106:
    /** SQL string if this is a SQL select file*/: First sentence should end 
with a period. [JavadocStyle]
   
./hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/impl/S3AOpenFileOperation.java:280:
  /**: First sentence should end with a period. [JavadocStyle]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/impl/TestOpenFileHelper.java:215:
  /**: First sentence should end with a period. [JavadocStyle]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/performance/ITestS3AOpenCost.java:169:
          byte[] out = new byte[(int) longLen];: 'block' child has incorrect 
indentation level 10, expected level should be 6. [Indentation]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/performance/ITestS3AOpenCost.java:170:
          intercept(EOFException.class,: 'block' child has incorrect 
indentation level 10, expected level should be 6. [Indentation]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/performance/ITestS3AOpenCost.java:172:
          in3.seek(longLen - 1);: 'block' child has incorrect indentation level 
10, expected level should be 6. [Indentation]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/performance/ITestS3AOpenCost.java:173:
          assertEquals("read past real EOF on " + in3,: 'block' child has 
incorrect indentation level 10, expected level should be 6. [Indentation]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/performance/ITestS3AOpenCost.java:175:
          in3.close();: 'block' child has incorrect indentation level 10, 
expected level should be 6. [Indentation]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/performance/ITestS3AOpenCost.java:176:
          return null;: 'block' child has incorrect indentation level 10, 
expected level should be 6. [Indentation]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/performance/ITestS3AOpenCost.java:177:
        },: 'block rcurly' has incorrect indentation level 8, expected level 
should be 4. [Indentation]
   ```


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



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

Reply via email to