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


   checkstyles are either unfixable (initialize()) or deliberate (test case 
naming)
   ```
   
./hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java:337:
  public void initialize(URI name, Configuration originalConf):3: Method length 
is 151 lines (max allowed is 150). [MethodLength]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/tools/ITestMarkerToolRootOperations.java:49:
  public void test_100_audit_root_noauth() throws Throwable {:15: Name 
'test_100_audit_root_noauth' must match pattern '^[a-z][a-zA-Z0-9]*$'. 
[MethodName]
   
./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/tools/ITestMarkerToolRootOperations.java:60:
  public void test_200_clean_root() throws Throwable {:15: Name 
'test_200_clean_root' must match pattern '^[a-z][a-zA-Z0-9]*$'. [MethodName]
   ```
   
   I'm going to merge this in now -just do one final review of the docs. I 
think I'll add a compatibility statement on the index.md page too, something to 
repeat for the other releases as I backport the getFileStatus change


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