hadoop-yetus removed a comment on issue #1820: HADOOP-16830. Add public 
IOStatistics API + S3A implementation
URL: https://github.com/apache/hadoop/pull/1820#issuecomment-588482038
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   1m 10s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files 
found.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 
12 new or modified test files.  |
   ||| _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 37s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  21m 58s |  trunk passed  |
   | +1 :green_heart: |  compile  |  19m 23s |  trunk passed  |
   | +1 :green_heart: |  checkstyle  |   2m 53s |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |   2m 10s |  trunk passed  |
   | +1 :green_heart: |  shadedclient  |  21m 57s |  branch has no errors when 
building and testing our client artifacts.  |
   | +1 :green_heart: |  javadoc  |   1m 26s |  trunk passed  |
   | +0 :ok: |  spotbugs  |   1m  7s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | +1 :green_heart: |  findbugs  |   3m 17s |  trunk passed  |
   | -0 :warning: |  patch  |   1m 29s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 25s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   1m 25s |  the patch passed  |
   | +1 :green_heart: |  compile  |  18m 11s |  the patch passed  |
   | +1 :green_heart: |  javac  |  18m 11s |  the patch passed  |
   | -0 :warning: |  checkstyle  |   2m 51s |  root: The patch generated 97 new 
+ 95 unchanged - 19 fixed = 192 total (was 114)  |
   | +1 :green_heart: |  mvnsite  |   2m  7s |  the patch passed  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  shadedclient  |  15m 24s |  patch has no errors when 
building and testing our client artifacts.  |
   | -1 :x: |  javadoc  |   0m 54s |  hadoop-common-project_hadoop-common 
generated 87 new + 101 unchanged - 0 fixed = 188 total (was 101)  |
   | -1 :x: |  findbugs  |   2m 18s |  hadoop-common-project/hadoop-common 
generated 29 new + 0 unchanged - 0 fixed = 29 total (was 0)  |
   | -1 :x: |  findbugs  |   1m 16s |  hadoop-tools/hadoop-aws generated 14 new 
+ 0 unchanged - 0 fixed = 14 total (was 0)  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   9m 32s |  hadoop-common in the patch passed. 
 |
   | +1 :green_heart: |  unit  |   1m 25s |  hadoop-aws in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   0m 45s |  The patch does not generate 
ASF License warnings.  |
   |  |   | 131m 46s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | FindBugs | module:hadoop-common-project/hadoop-common |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
42] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
45] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
48] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
51] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
54] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
57] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
60] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
63] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
66] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
69] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
72] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
75] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
81] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
78] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
84] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
87] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
90] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
93] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
96] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
99] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
102] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
105] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
108] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
111] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
114] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
117] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
120] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
123] |
   |  |  Unread public/protected field:At FilesystemStatisticNames.java:[line 
126] |
   | FindBugs | module:hadoop-tools/hadoop-aws |
   |  |  Dead store to ioStats in new 
org.apache.hadoop.fs.s3a.S3AInputStream(S3AReadOpContext, S3ObjectAttributes, 
AmazonS3)  At S3AInputStream.java:new 
org.apache.hadoop.fs.s3a.S3AInputStream(S3AReadOpContext, S3ObjectAttributes, 
AmazonS3)  At S3AInputStream.java:[line 175] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.policySetCount
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.inputPolicySet(int)
  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.inputPolicySet(int)
  At S3AInstrumentation.java:[line 804] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readExceptions
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readException()
  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readException()
  At S3AInstrumentation.java:[line 741] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readFullyOperations
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readFullyOperationStarted(long,
 long)  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readFullyOperationStarted(long,
 long)  At S3AInstrumentation.java:[line 774] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readsIncomplete
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readOperationCompleted(int,
 int)  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readOperationCompleted(int,
 int)  At S3AInstrumentation.java:[line 785] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readOperations
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readOperationStarted(long,
 long)  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.readOperationStarted(long,
 long)  At S3AInstrumentation.java:[line 763] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.backwardSeekOperations
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekBackwards(long)
  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekBackwards(long)
  At S3AInstrumentation.java:[line 686] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekOperations
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekBackwards(long)
  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekBackwards(long)
  At S3AInstrumentation.java:[line 685] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.forwardSeekOperations
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekForwards(long)
  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekForwards(long)
  At S3AInstrumentation.java:[line 699] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekOperations
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekForwards(long)
  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.seekForwards(long)
  At S3AInstrumentation.java:[line 698] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.aborted 
in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.streamClose(boolean,
 long)  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.streamClose(boolean,
 long)  At S3AInstrumentation.java:[line 728] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.closeOperations
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.streamClose(boolean,
 long)  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.streamClose(boolean,
 long)  At S3AInstrumentation.java:[line 726] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.closed in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.streamClose(boolean,
 long)  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.streamClose(boolean,
 long)  At S3AInstrumentation.java:[line 731] |
   |  |  Increment of volatile field 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.openOperations
 in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.streamOpened()
  At S3AInstrumentation.java:in 
org.apache.hadoop.fs.s3a.S3AInstrumentation$InputStreamStatisticsImpl.streamOpened()
  At S3AInstrumentation.java:[line 712] |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | Client=19.03.6 Server=19.03.6 base: 
https://builds.apache.org/job/hadoop-multibranch/job/PR-1820/5/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hadoop/pull/1820 |
   | Optional Tests | dupname asflicense compile javac javadoc mvninstall 
mvnsite unit shadedclient findbugs checkstyle |
   | uname | Linux 3be3de3f69ff 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 
08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | personality/hadoop.sh |
   | git revision | trunk / 3f1aad0 |
   | Default Java | 1.8.0_242 |
   | checkstyle | 
https://builds.apache.org/job/hadoop-multibranch/job/PR-1820/5/artifact/out/diff-checkstyle-root.txt
 |
   | javadoc | 
https://builds.apache.org/job/hadoop-multibranch/job/PR-1820/5/artifact/out/diff-javadoc-javadoc-hadoop-common-project_hadoop-common.txt
 |
   | findbugs | 
https://builds.apache.org/job/hadoop-multibranch/job/PR-1820/5/artifact/out/new-findbugs-hadoop-common-project_hadoop-common.html
 |
   | findbugs | 
https://builds.apache.org/job/hadoop-multibranch/job/PR-1820/5/artifact/out/new-findbugs-hadoop-tools_hadoop-aws.html
 |
   |  Test Results | 
https://builds.apache.org/job/hadoop-multibranch/job/PR-1820/5/testReport/ |
   | Max. process+thread count | 1347 (vs. ulimit of 5500) |
   | modules | C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws 
U: . |
   | Console output | 
https://builds.apache.org/job/hadoop-multibranch/job/PR-1820/5/console |
   | versions | git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1 |
   | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   

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


With regards,
Apache Git Services

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