hadoop-yetus commented on PR #6633:
URL: https://github.com/apache/hadoop/pull/6633#issuecomment-2006936579

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |:----:|----------:|--------:|:--------:|:-------:|
   | +0 :ok: |  reexec  |   0m 33s |  |  Docker mode activated.  |
   |||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  0s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +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 18 new or modified test files.  |
   |||| _ trunk Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |  45m 20s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |   0m 40s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  compile  |   0m 34s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   0m 31s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |   0m 41s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   0m 37s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javadoc  |   0m 33s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  spotbugs  |   1m  7s |  |  trunk passed  |
   | +1 :green_heart: |  shadedclient  |  35m  1s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  35m 22s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   |||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   0m 29s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 31s |  |  the patch passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javac  |   0m 31s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 27s |  |  the patch passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  javac  |   0m 27s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   0m 21s | 
[/results-checkstyle-hadoop-tools_hadoop-azure.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/6/artifact/out/results-checkstyle-hadoop-tools_hadoop-azure.txt)
 |  hadoop-tools/hadoop-azure: The patch generated 134 new + 18 unchanged - 0 
fixed = 152 total (was 18)  |
   | +1 :green_heart: |  mvnsite  |   0m 32s |  |  the patch passed  |
   | -1 :x: |  javadoc  |   0m 27s | 
[/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkUbuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/6/artifact/out/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkUbuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1.txt)
 |  hadoop-tools_hadoop-azure-jdkUbuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 
with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 generated 3 new + 15 
unchanged - 0 fixed = 18 total (was 15)  |
   | -1 :x: |  javadoc  |   0m 26s | 
[/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_392-8u392-ga-1~20.04-b08.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/6/artifact/out/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_392-8u392-ga-1~20.04-b08.txt)
 |  hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_392-8u392-ga-1~20.04-b08 
with JDK Private Build-1.8.0_392-8u392-ga-1~20.04-b08 generated 3 new + 15 
unchanged - 0 fixed = 18 total (was 15)  |
   | -1 :x: |  spotbugs  |   1m 12s | 
[/new-spotbugs-hadoop-tools_hadoop-azure.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/6/artifact/out/new-spotbugs-hadoop-tools_hadoop-azure.html)
 |  hadoop-tools/hadoop-azure generated 18 new + 0 unchanged - 0 fixed = 18 
total (was 0)  |
   | +1 :green_heart: |  shadedclient  |  34m 42s |  |  patch has no errors 
when building and testing our client artifacts.  |
   |||| _ Other Tests _ |
   | -1 :x: |  unit  |   2m 27s | 
[/patch-unit-hadoop-tools_hadoop-azure.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/6/artifact/out/patch-unit-hadoop-tools_hadoop-azure.txt)
 |  hadoop-azure in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   0m 37s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   | 132m 21s |  |  |
   
   
   | Reason | Tests |
   |-------:|:------|
   | SpotBugs | module:hadoop-tools/hadoop-azure |
   |  |  Unread field:AbfsConnectionManager.java:[line 113] |
   |  |  Unread field:AbfsApacheHttpClient.java:[line 63] |
   |  |  Unread field:AbfsApacheHttpClient.java:[line 88] |
   |  |  Unread field:AbfsApacheHttpClient.java:[line 68] |
   |  |  Unused field:AbfsApacheHttpClient.java |
   |  |  Unused field:AbfsApacheHttpClient.java |
   |  |  Dead store to start in 
org.apache.hadoop.fs.azurebfs.services.AbfsConnFactory$AbfsApacheHttpConnection.isResponseAvailable(int)
  At 
AbfsConnFactory.java:org.apache.hadoop.fs.azurebfs.services.AbfsConnFactory$AbfsApacheHttpConnection.isResponseAvailable(int)
  At AbfsConnFactory.java:[line 92] |
   |  |  Dead store to start in 
org.apache.hadoop.fs.azurebfs.services.AbfsConnFactory$AbfsApacheHttpConnection.receiveResponseHeader()
  At 
AbfsConnFactory.java:org.apache.hadoop.fs.azurebfs.services.AbfsConnFactory$AbfsApacheHttpConnection.receiveResponseHeader()
  At AbfsConnFactory.java:[line 113] |
   |  |  Dead store to start in 
org.apache.hadoop.fs.azurebfs.services.AbfsConnFactory$AbfsApacheHttpConnection.sendRequestHeader(HttpRequest)
  At 
AbfsConnFactory.java:org.apache.hadoop.fs.azurebfs.services.AbfsConnFactory$AbfsApacheHttpConnection.sendRequestHeader(HttpRequest)
  At AbfsConnFactory.java:[line 100] |
   |  |  Dead store to startTime in 
org.apache.hadoop.fs.azurebfs.services.AbfsHttpOperation.processConnHeadersAndInputStreams(byte[],
 int, int)  At 
AbfsHttpOperation.java:org.apache.hadoop.fs.azurebfs.services.AbfsHttpOperation.processConnHeadersAndInputStreams(byte[],
 int, int)  At AbfsHttpOperation.java:[line 337] |
   |  |  org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache.INSTANCE 
isn't final and can't be protected from malicious code  At 
KeepAliveCache.java:be protected from malicious code  At 
KeepAliveCache.java:[line 71] |
   |  |  Exception is caught when Exception is not thrown in 
org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache.kacCleanup()  At 
KeepAliveCache.java:is not thrown in 
org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache.kacCleanup()  At 
KeepAliveCache.java:[line 131] |
   |  |  Class org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache 
defines non-transient non-serializable instance field thread  In 
KeepAliveCache.java:instance field thread  In KeepAliveCache.java |
   |  |  Write to static field 
org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache.INSTANCE from 
instance method 
org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache.close()  At 
KeepAliveCache.java:from instance method 
org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache.close()  At 
KeepAliveCache.java:[line 47] |
   |  |  org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache.kacCleanup() 
makes inefficient use of keySet iterator instead of entrySet iterator  At 
KeepAliveCache.java:keySet iterator instead of entrySet iterator  At 
KeepAliveCache.java:[line 106] |
   |  |  org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache$ClientVector 
doesn't override java.util.Vector.equals(Object)  At KeepAliveCache.java:At 
KeepAliveCache.java:[line 1] |
   |  |  Should 
org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache$KeepAliveEntry be a 
_static_ inner class?  At KeepAliveCache.java:inner class?  At 
KeepAliveCache.java:[lines 247-250] |
   |  |  Should 
org.apache.hadoop.fs.azurebfs.services.kac.KeepAliveCache$KeepAliveKey be a 
_static_ inner class?  At KeepAliveCache.java:inner class?  At 
KeepAliveCache.java:[lines 220-239] |
   | Failed junit tests | 
hadoop.fs.azurebfs.services.TestApacheHttpClientFallback |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.44 ServerAPI=1.44 base: 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/6/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hadoop/pull/6633 |
   | Optional Tests | dupname asflicense compile javac javadoc mvninstall 
mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
   | uname | Linux 5d5a7355a53d 5.15.0-94-generic #104-Ubuntu SMP Tue Jan 9 
15:25:40 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/bin/hadoop.sh |
   | git revision | trunk / 665adada2a48d9faff8a3f35e8885090c32391aa |
   | Default Java | Private Build-1.8.0_392-8u392-ga-1~20.04-b08 |
   | Multi-JDK versions | 
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 
/usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_392-8u392-ga-1~20.04-b08 |
   |  Test Results | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/6/testReport/ |
   | Max. process+thread count | 551 (vs. ulimit of 5500) |
   | modules | C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure |
   | Console output | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/6/console |
   | versions | git=2.25.1 maven=3.6.3 spotbugs=4.2.2 |
   | Powered by | Apache Yetus 0.14.0 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.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

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