hadoop-yetus commented on PR #6633: URL: https://github.com/apache/hadoop/pull/6633#issuecomment-2006966559
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |:----:|----------:|--------:|:--------:|:-------:| | +0 :ok: | reexec | 0m 31s | | 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 | 44m 23s | | trunk passed | | +1 :green_heart: | compile | 0m 38s | | trunk passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 | | +1 :green_heart: | compile | 0m 36s | | trunk passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06 | | +1 :green_heart: | checkstyle | 0m 32s | | trunk passed | | +1 :green_heart: | mvnsite | 0m 40s | | trunk passed | | +1 :green_heart: | javadoc | 0m 39s | | trunk passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 | | +1 :green_heart: | javadoc | 0m 36s | | trunk passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06 | | +1 :green_heart: | spotbugs | 1m 7s | | trunk passed | | +1 :green_heart: | shadedclient | 33m 18s | | branch has no errors when building and testing our client artifacts. | | -0 :warning: | patch | 33m 39s | | 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 30s | | the patch passed | | +1 :green_heart: | compile | 0m 30s | | the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 | | +1 :green_heart: | javac | 0m 30s | | the patch passed | | +1 :green_heart: | compile | 0m 27s | | the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06 | | +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/7/artifact/out/results-checkstyle-hadoop-tools_hadoop-azure.txt) | hadoop-tools/hadoop-azure: The patch generated 136 new + 18 unchanged - 0 fixed = 154 total (was 18) | | +1 :green_heart: | mvnsite | 0m 31s | | the patch passed | | -1 :x: | javadoc | 0m 26s | [/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/7/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 27s | [/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/7/artifact/out/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06.txt) | hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06 with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06 generated 3 new + 15 unchanged - 0 fixed = 18 total (was 15) | | -1 :x: | spotbugs | 1m 7s | [/new-spotbugs-hadoop-tools_hadoop-azure.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/7/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 | 33m 18s | | patch has no errors when building and testing our client artifacts. | |||| _ Other Tests _ | | +1 :green_heart: | unit | 2m 26s | | hadoop-azure in the patch passed. | | +1 :green_heart: | asflicense | 0m 38s | | The patch does not generate ASF License warnings. | | | | 128m 7s | | | | 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] | | Subsystem | Report/Notes | |----------:|:-------------| | Docker | ClientAPI=1.44 ServerAPI=1.44 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/7/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 0590ec9e850e 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 / 53348ffd5b1f3102e9dda16b2f5fa2fda64b82b8 | | Default Java | Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06 | | 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_402-8u402-ga-2ubuntu1~20.04-b06 | | Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/7/testReport/ | | Max. process+thread count | 763 (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/7/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