hadoop-yetus commented on PR #6633: URL: https://github.com/apache/hadoop/pull/6633#issuecomment-2009123337
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |:----:|----------:|--------:|:--------:|:-------:| | +0 :ok: | reexec | 0m 32s | | 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 19 new or modified test files. | |||| _ trunk Compile Tests _ | | +1 :green_heart: | mvninstall | 43m 50s | | trunk passed | | +1 :green_heart: | compile | 0m 39s | | trunk passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 | | +1 :green_heart: | compile | 0m 37s | | trunk passed with JDK Private Build-1.8.0_392-8u392-ga-1~20.04-b08 | | +1 :green_heart: | checkstyle | 0m 33s | | trunk passed | | +1 :green_heart: | mvnsite | 0m 41s | | trunk passed | | +1 :green_heart: | javadoc | 0m 40s | | 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_392-8u392-ga-1~20.04-b08 | | +1 :green_heart: | spotbugs | 1m 7s | | trunk passed | | +1 :green_heart: | shadedclient | 32m 46s | | branch has no errors when building and testing our client artifacts. | | -0 :warning: | patch | 33m 7s | | 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 29s | | the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 | | +1 :green_heart: | javac | 0m 29s | | 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/12/artifact/out/results-checkstyle-hadoop-tools_hadoop-azure.txt) | hadoop-tools/hadoop-azure: The patch generated 135 new + 18 unchanged - 0 fixed = 153 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/12/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 5 new + 15 unchanged - 0 fixed = 20 total (was 15) | | -1 :x: | javadoc | 0m 25s | [/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/12/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 5 new + 15 unchanged - 0 fixed = 20 total (was 15) | | -1 :x: | spotbugs | 1m 10s | [/new-spotbugs-hadoop-tools_hadoop-azure.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6633/12/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 | 32m 41s | | patch has no errors when building and testing our client artifacts. | |||| _ Other Tests _ | | +1 :green_heart: | unit | 2m 27s | | hadoop-azure in the patch passed. | | +1 :green_heart: | asflicense | 0m 38s | | The patch does not generate ASF License warnings. | | | | 126m 38s | | | | Reason | Tests | |-------:|:------| | SpotBugs | module:hadoop-tools/hadoop-azure | | | 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] | | | Dead store to start in org.apache.hadoop.fs.azurebfs.services.AbfsManagedApacheHttpConnection.isResponseAvailable(int) At AbfsManagedApacheHttpConnection.java:org.apache.hadoop.fs.azurebfs.services.AbfsManagedApacheHttpConnection.isResponseAvailable(int) At AbfsManagedApacheHttpConnection.java:[line 84] | | | Dead store to start in org.apache.hadoop.fs.azurebfs.services.AbfsManagedApacheHttpConnection.receiveResponseHeader() At AbfsManagedApacheHttpConnection.java:org.apache.hadoop.fs.azurebfs.services.AbfsManagedApacheHttpConnection.receiveResponseHeader() At AbfsManagedApacheHttpConnection.java:[line 105] | | | Dead store to start in org.apache.hadoop.fs.azurebfs.services.AbfsManagedApacheHttpConnection.sendRequestHeader(HttpRequest) At AbfsManagedApacheHttpConnection.java:org.apache.hadoop.fs.azurebfs.services.AbfsManagedApacheHttpConnection.sendRequestHeader(HttpRequest) At AbfsManagedApacheHttpConnection.java:[line 92] | | | Unread field:AbfsConnectionManager.java:[line 116] | | | Unread field:AbfsManagedHttpContext.java:[line 34] | | | Unread field:AbfsManagedHttpRequestExecutor.java:[line 57] | | | Unread field:AbfsManagedHttpContext.java:[line 40] | | | Unused field:AbfsManagedHttpContext.java | | | Unused field:AbfsManagedHttpContext.java | | | 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/12/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 7cdcfeb86384 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 / 3cb52f4c1c3bb146a6762a6a279f83866cd9b249 | | 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/12/testReport/ | | Max. process+thread count | 552 (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/12/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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
