hadoop
hadoop copied to clipboard
HADOOP-18324. Interrupting RPC Client calls can lead to thread exhaustion.
Description of PR
I modified the RPC Connection class to use a single thread that does the writes to the socket instead of a thread factory. (It already has a different thread that does the reads.) Java’s concurrency library has a SynchronousQueue that will simplify hand offs from the calling thread to the rpc sending thread.
As a result, we’ll end up with:
- Exactly 1 sending thread per an RPC connection.
- If the calling thread is interrupted before the socket write, it will be skipped instead of sending it anyways.
- If the calling thread is interrupted during the socket write, the write will finish.
- RPC requests will be written to the socket in the order received.
How was this patch tested?
Added a unit test.
:broken_heart: -1 overall
Vote | Subsystem | Runtime | Logfile | Comment |
---|---|---|---|---|
+0 :ok: | reexec | 0m 56s | Docker mode activated. | |
_ Prechecks _ | ||||
+1 :green_heart: | dupname | 0m 0s | 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 2 new or modified test files. | |
_ trunk Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 37m 53s | trunk passed | |
+1 :green_heart: | compile | 22m 59s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | compile | 20m 24s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | checkstyle | 1m 50s | trunk passed | |
+1 :green_heart: | mvnsite | 2m 15s | trunk passed | |
+1 :green_heart: | javadoc | 1m 49s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 49s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 8s | trunk passed | |
+1 :green_heart: | shadedclient | 23m 40s | branch has no errors when building and testing our client artifacts. | |
_ Patch Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 1m 6s | the patch passed | |
+1 :green_heart: | compile | 22m 20s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
-1 :x: | javac | 22m 20s | /results-compile-javac-root-jdkPrivateBuild-11.0.15+10-Ubuntu-0ubuntu0.20.04.1.txt | root-jdkPrivateBuild-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 generated 1 new + 2879 unchanged - 0 fixed = 2880 total (was 2879) |
+1 :green_heart: | compile | 20m 43s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
-1 :x: | javac | 20m 43s | /results-compile-javac-root-jdkPrivateBuild-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07.txt | root-jdkPrivateBuild-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 generated 1 new + 2676 unchanged - 0 fixed = 2677 total (was 2676) |
+1 :green_heart: | blanks | 0m 0s | The patch has no blanks issues. | |
-0 :warning: | checkstyle | 1m 41s | /results-checkstyle-hadoop-common-project_hadoop-common.txt | hadoop-common-project/hadoop-common: The patch generated 7 new + 147 unchanged - 3 fixed = 154 total (was 150) |
+1 :green_heart: | mvnsite | 2m 10s | the patch passed | |
+1 :green_heart: | javadoc | 1m 39s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 20s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 9s | the patch passed | |
+1 :green_heart: | shadedclient | 23m 16s | patch has no errors when building and testing our client artifacts. | |
_ Other Tests _ | ||||
+1 :green_heart: | unit | 18m 40s | hadoop-common in the patch passed. | |
+1 :green_heart: | asflicense | 1m 36s | The patch does not generate ASF License warnings. | |
215m 56s |
Subsystem | Report/Notes |
---|---|
Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/1/artifact/out/Dockerfile |
GITHUB PR | https://github.com/apache/hadoop/pull/4527 |
Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
uname | Linux 543501ffccd4 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
Build tool | maven |
Personality | dev-support/bin/hadoop.sh |
git revision | trunk / 6a52d291df2bc3ea50a7aa796398d29bc42a2ac3 |
Default Java | Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/1/testReport/ |
Max. process+thread count | 1341 (vs. ulimit of 5500) |
modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/1/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.
@omalley I'm very happy that you noticing this issue and working on it. This problem occurs frequently in out prod environment when the network is abnormal. About this rpc sending model, I have one question and looking forward your feedback.
- What's the underlying idea for using asynchronous threads to send request synchronously?
In addition to guaranteeing that only one request is sent at the same time, does it need to guarantee the order of sending?
If it is just to ensure mutual exclusive transmission, can we do it by a sync lock? And if so, we can use the business thread to send it's own request directly. And we can remove the send thread in Client.java
.
The extra thread is used so that if the calling thread is interrupted while the request is being written to the socket, that the write is not interrupted. Basically, it is ensuring that the socket doesn't have partial requests sent to it.
Without that, Hadoop would need to resynchronize by closing the socket.
My patch also guarantees that the requests are written in the order they arrived, which the current code does not.
The extra thread is used so that if the calling thread is interrupted while the request is being written to the socket, that the write is not interrupted
Nice explain, I learned a log. Thanks very much.
+1 (non-binding). lets make the checkstyle happy as well
@steveloughran Can you recommend someone to review this RPC patch?
:broken_heart: -1 overall
Vote | Subsystem | Runtime | Logfile | Comment |
---|---|---|---|---|
+0 :ok: | reexec | 0m 52s | Docker mode activated. | |
_ Prechecks _ | ||||
+1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | |
+0 :ok: | codespell | 0m 1s | codespell was not available. | |
+0 :ok: | detsecrets | 0m 1s | 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 2 new or modified test files. | |
_ trunk Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 37m 38s | trunk passed | |
+1 :green_heart: | compile | 24m 32s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | compile | 23m 1s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | checkstyle | 1m 52s | trunk passed | |
+1 :green_heart: | mvnsite | 2m 13s | trunk passed | |
+1 :green_heart: | javadoc | 1m 48s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 25s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 12s | trunk passed | |
+1 :green_heart: | shadedclient | 23m 29s | branch has no errors when building and testing our client artifacts. | |
_ Patch Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 1m 6s | the patch passed | |
+1 :green_heart: | compile | 22m 10s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
-1 :x: | javac | 22m 10s | /results-compile-javac-root-jdkPrivateBuild-11.0.15+10-Ubuntu-0ubuntu0.20.04.1.txt | root-jdkPrivateBuild-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 generated 1 new + 2879 unchanged - 0 fixed = 2880 total (was 2879) |
+1 :green_heart: | compile | 20m 27s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
-1 :x: | javac | 20m 27s | /results-compile-javac-root-jdkPrivateBuild-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07.txt | root-jdkPrivateBuild-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 generated 1 new + 2676 unchanged - 0 fixed = 2677 total (was 2676) |
+1 :green_heart: | blanks | 0m 0s | The patch has no blanks issues. | |
-0 :warning: | checkstyle | 1m 43s | /results-checkstyle-hadoop-common-project_hadoop-common.txt | hadoop-common-project/hadoop-common: The patch generated 1 new + 147 unchanged - 3 fixed = 148 total (was 150) |
+1 :green_heart: | mvnsite | 2m 11s | the patch passed | |
+1 :green_heart: | javadoc | 1m 39s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 23s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 13s | the patch passed | |
+1 :green_heart: | shadedclient | 23m 35s | patch has no errors when building and testing our client artifacts. | |
_ Other Tests _ | ||||
+1 :green_heart: | unit | 18m 49s | hadoop-common in the patch passed. | |
+1 :green_heart: | asflicense | 1m 35s | The patch does not generate ASF License warnings. | |
219m 20s |
Subsystem | Report/Notes |
---|---|
Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/2/artifact/out/Dockerfile |
GITHUB PR | https://github.com/apache/hadoop/pull/4527 |
Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
uname | Linux 63998bedd3c4 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
Build tool | maven |
Personality | dev-support/bin/hadoop.sh |
git revision | trunk / b39b63eabe2ddc4b508af89c9ef8f00f65439213 |
Default Java | Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/2/testReport/ |
Max. process+thread count | 3152 (vs. ulimit of 5500) |
modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/2/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.
Ask on the HDEFS mailing list; they are the ones that get unhappy first when things break.
:confetti_ball: +1 overall
Vote | Subsystem | Runtime | Logfile | Comment |
---|---|---|---|---|
+0 :ok: | reexec | 0m 51s | Docker mode activated. | |
_ Prechecks _ | ||||
+1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | |
+0 :ok: | codespell | 0m 1s | codespell was not available. | |
+0 :ok: | detsecrets | 0m 1s | 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 2 new or modified test files. | |
_ trunk Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 37m 37s | trunk passed | |
+1 :green_heart: | compile | 22m 51s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | compile | 20m 26s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | checkstyle | 1m 46s | trunk passed | |
+1 :green_heart: | mvnsite | 2m 11s | trunk passed | |
+1 :green_heart: | javadoc | 1m 49s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 21s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 17s | trunk passed | |
+1 :green_heart: | shadedclient | 23m 39s | branch has no errors when building and testing our client artifacts. | |
_ Patch Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 1m 6s | the patch passed | |
+1 :green_heart: | compile | 22m 8s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javac | 22m 8s | the patch passed | |
+1 :green_heart: | compile | 20m 33s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | javac | 20m 33s | the patch passed | |
+1 :green_heart: | blanks | 0m 0s | The patch has no blanks issues. | |
-0 :warning: | checkstyle | 1m 43s | /results-checkstyle-hadoop-common-project_hadoop-common.txt | hadoop-common-project/hadoop-common: The patch generated 1 new + 147 unchanged - 3 fixed = 148 total (was 150) |
+1 :green_heart: | mvnsite | 2m 11s | the patch passed | |
+1 :green_heart: | javadoc | 1m 39s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 23s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 14s | the patch passed | |
+1 :green_heart: | shadedclient | 26m 58s | patch has no errors when building and testing our client artifacts. | |
_ Other Tests _ | ||||
+1 :green_heart: | unit | 19m 12s | hadoop-common in the patch passed. | |
+1 :green_heart: | asflicense | 1m 21s | The patch does not generate ASF License warnings. | |
218m 41s |
Subsystem | Report/Notes |
---|---|
Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/3/artifact/out/Dockerfile |
GITHUB PR | https://github.com/apache/hadoop/pull/4527 |
Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
uname | Linux 93e4f459fc7c 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
Build tool | maven |
Personality | dev-support/bin/hadoop.sh |
git revision | trunk / 971cac305447d94a134bc7faf562c1f92fff645c |
Default Java | Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/3/testReport/ |
Max. process+thread count | 3152 (vs. ulimit of 5500) |
modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/3/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.
:confetti_ball: +1 overall
Vote | Subsystem | Runtime | Logfile | Comment |
---|---|---|---|---|
+0 :ok: | reexec | 0m 59s | Docker mode activated. | |
_ Prechecks _ | ||||
+1 :green_heart: | dupname | 0m 0s | 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 2 new or modified test files. | |
_ trunk Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 37m 23s | trunk passed | |
+1 :green_heart: | compile | 23m 4s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | compile | 20m 39s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | checkstyle | 1m 48s | trunk passed | |
+1 :green_heart: | mvnsite | 2m 9s | trunk passed | |
+1 :green_heart: | javadoc | 1m 51s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 23s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 19s | trunk passed | |
+1 :green_heart: | shadedclient | 23m 44s | branch has no errors when building and testing our client artifacts. | |
_ Patch Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 1m 6s | the patch passed | |
+1 :green_heart: | compile | 22m 28s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javac | 22m 28s | the patch passed | |
+1 :green_heart: | compile | 20m 36s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | javac | 20m 36s | the patch passed | |
+1 :green_heart: | blanks | 0m 0s | The patch has no blanks issues. | |
+1 :green_heart: | checkstyle | 1m 43s | hadoop-common-project/hadoop-common: The patch generated 0 new + 147 unchanged - 3 fixed = 147 total (was 150) | |
+1 :green_heart: | mvnsite | 2m 12s | the patch passed | |
+1 :green_heart: | javadoc | 1m 40s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 25s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 10s | the patch passed | |
+1 :green_heart: | shadedclient | 23m 30s | patch has no errors when building and testing our client artifacts. | |
_ Other Tests _ | ||||
+1 :green_heart: | unit | 18m 49s | hadoop-common in the patch passed. | |
+1 :green_heart: | asflicense | 1m 30s | The patch does not generate ASF License warnings. | |
215m 53s |
Subsystem | Report/Notes |
---|---|
Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/4/artifact/out/Dockerfile |
GITHUB PR | https://github.com/apache/hadoop/pull/4527 |
Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
uname | Linux edc1e55ad849 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
Build tool | maven |
Personality | dev-support/bin/hadoop.sh |
git revision | trunk / 04d63ab73bea17d5051c4c5494b5ea55b6741da1 |
Default Java | Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/4/testReport/ |
Max. process+thread count | 3158 (vs. ulimit of 5500) |
modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/4/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.
:confetti_ball: +1 overall
Vote | Subsystem | Runtime | Logfile | Comment |
---|---|---|---|---|
+0 :ok: | reexec | 1m 10s | Docker mode activated. | |
_ Prechecks _ | ||||
+1 :green_heart: | dupname | 0m 0s | 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 2 new or modified test files. | |
_ trunk Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 38m 55s | trunk passed | |
+1 :green_heart: | compile | 25m 46s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | compile | 22m 26s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | checkstyle | 1m 42s | trunk passed | |
+1 :green_heart: | mvnsite | 2m 8s | trunk passed | |
+1 :green_heart: | javadoc | 1m 46s | trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 19s | trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 17s | trunk passed | |
+1 :green_heart: | shadedclient | 25m 42s | branch has no errors when building and testing our client artifacts. | |
_ Patch Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 1m 10s | the patch passed | |
+1 :green_heart: | compile | 24m 22s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javac | 24m 22s | the patch passed | |
+1 :green_heart: | compile | 22m 14s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | javac | 22m 14s | the patch passed | |
+1 :green_heart: | blanks | 0m 0s | The patch has no blanks issues. | |
+1 :green_heart: | checkstyle | 1m 41s | hadoop-common-project/hadoop-common: The patch generated 0 new + 147 unchanged - 3 fixed = 147 total (was 150) | |
+1 :green_heart: | mvnsite | 2m 14s | the patch passed | |
+1 :green_heart: | javadoc | 1m 35s | the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 | |
+1 :green_heart: | javadoc | 1m 14s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 19s | the patch passed | |
+1 :green_heart: | shadedclient | 25m 49s | patch has no errors when building and testing our client artifacts. | |
_ Other Tests _ | ||||
+1 :green_heart: | unit | 19m 17s | hadoop-common in the patch passed. | |
+1 :green_heart: | asflicense | 1m 29s | The patch does not generate ASF License warnings. | |
229m 34s |
Subsystem | Report/Notes |
---|---|
Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/5/artifact/out/Dockerfile |
GITHUB PR | https://github.com/apache/hadoop/pull/4527 |
Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
uname | Linux c552356985f5 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
Build tool | maven |
Personality | dev-support/bin/hadoop.sh |
git revision | trunk / 38169e5adcbdfe5fd8d15bcf7fb98982e5c27e36 |
Default Java | Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/5/testReport/ |
Max. process+thread count | 3158 (vs. ulimit of 5500) |
modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/5/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.
Sorry, I got busy with other work. I've included fixes that we've deployed for a while now. I had missed some cases of IOException handling and start up issues.
:confetti_ball: +1 overall
Vote | Subsystem | Runtime | Logfile | Comment |
---|---|---|---|---|
+0 :ok: | reexec | 0m 56s | Docker mode activated. | |
_ Prechecks _ | ||||
+1 :green_heart: | dupname | 0m 0s | 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 2 new or modified test files. | |
_ trunk Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 40m 58s | trunk passed | |
+1 :green_heart: | compile | 23m 28s | trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | compile | 20m 48s | trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | checkstyle | 1m 29s | trunk passed | |
+1 :green_heart: | mvnsite | 2m 0s | trunk passed | |
+1 :green_heart: | javadoc | 1m 48s | trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javadoc | 1m 11s | trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 5s | trunk passed | |
+1 :green_heart: | shadedclient | 23m 16s | branch has no errors when building and testing our client artifacts. | |
_ Patch Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 1m 6s | the patch passed | |
+1 :green_heart: | compile | 22m 34s | the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javac | 22m 34s | the patch passed | |
+1 :green_heart: | compile | 20m 52s | the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | javac | 20m 52s | the patch passed | |
+1 :green_heart: | blanks | 0m 0s | The patch has no blanks issues. | |
-0 :warning: | checkstyle | 1m 30s | /results-checkstyle-hadoop-common-project_hadoop-common.txt | hadoop-common-project/hadoop-common: The patch generated 2 new + 145 unchanged - 3 fixed = 147 total (was 148) |
+1 :green_heart: | mvnsite | 1m 56s | the patch passed | |
+1 :green_heart: | javadoc | 1m 19s | the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javadoc | 1m 1s | the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 2m 52s | the patch passed | |
+1 :green_heart: | shadedclient | 22m 48s | patch has no errors when building and testing our client artifacts. | |
_ Other Tests _ | ||||
+1 :green_heart: | unit | 18m 54s | hadoop-common in the patch passed. | |
+1 :green_heart: | asflicense | 1m 20s | The patch does not generate ASF License warnings. | |
216m 2s |
Subsystem | Report/Notes |
---|---|
Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/6/artifact/out/Dockerfile |
GITHUB PR | https://github.com/apache/hadoop/pull/4527 |
Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
uname | Linux 62eeed87afd0 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
Build tool | maven |
Personality | dev-support/bin/hadoop.sh |
git revision | trunk / 4df3227f6da08c8834dd9774a11a43d43f2d496d |
Default Java | Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 |
Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 |
Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/6/testReport/ |
Max. process+thread count | 1533 (vs. ulimit of 5500) |
modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/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.
:confetti_ball: +1 overall
Vote | Subsystem | Runtime | Logfile | Comment |
---|---|---|---|---|
+0 :ok: | reexec | 0m 49s | Docker mode activated. | |
_ Prechecks _ | ||||
+1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | |
+0 :ok: | codespell | 0m 1s | codespell was not available. | |
+0 :ok: | detsecrets | 0m 1s | 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 2 new or modified test files. | |
_ trunk Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 41m 16s | trunk passed | |
+1 :green_heart: | compile | 23m 27s | trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | compile | 21m 5s | trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | checkstyle | 1m 35s | trunk passed | |
+1 :green_heart: | mvnsite | 1m 53s | trunk passed | |
+1 :green_heart: | javadoc | 1m 30s | trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javadoc | 1m 21s | trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 15s | trunk passed | |
+1 :green_heart: | shadedclient | 23m 25s | branch has no errors when building and testing our client artifacts. | |
_ Patch Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 1m 4s | the patch passed | |
+1 :green_heart: | compile | 22m 34s | the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javac | 22m 34s | the patch passed | |
+1 :green_heart: | compile | 21m 5s | the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | javac | 21m 5s | the patch passed | |
+1 :green_heart: | blanks | 0m 0s | The patch has no blanks issues. | |
-0 :warning: | checkstyle | 1m 21s | /results-checkstyle-hadoop-common-project_hadoop-common.txt | hadoop-common-project/hadoop-common: The patch generated 2 new + 145 unchanged - 3 fixed = 147 total (was 148) |
+1 :green_heart: | mvnsite | 2m 0s | the patch passed | |
+1 :green_heart: | javadoc | 1m 17s | the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javadoc | 1m 20s | the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 2m 58s | the patch passed | |
+1 :green_heart: | shadedclient | 23m 16s | patch has no errors when building and testing our client artifacts. | |
_ Other Tests _ | ||||
+1 :green_heart: | unit | 18m 44s | hadoop-common in the patch passed. | |
+1 :green_heart: | asflicense | 1m 32s | The patch does not generate ASF License warnings. | |
217m 1s |
Subsystem | Report/Notes |
---|---|
Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/7/artifact/out/Dockerfile |
GITHUB PR | https://github.com/apache/hadoop/pull/4527 |
Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
uname | Linux d3c6a442d0d3 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
Build tool | maven |
Personality | dev-support/bin/hadoop.sh |
git revision | trunk / 4664aeeae276a2cde823ed63c14d8449878f368b |
Default Java | Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 |
Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 |
Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/7/testReport/ |
Max. process+thread count | 2497 (vs. ulimit of 5500) |
modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/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.
:confetti_ball: +1 overall
Vote | Subsystem | Runtime | Logfile | Comment |
---|---|---|---|---|
+0 :ok: | reexec | 1m 1s | Docker mode activated. | |
_ Prechecks _ | ||||
+1 :green_heart: | dupname | 0m 0s | 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 2 new or modified test files. | |
_ trunk Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 38m 36s | trunk passed | |
+1 :green_heart: | compile | 23m 30s | trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | compile | 20m 58s | trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | checkstyle | 1m 36s | trunk passed | |
+1 :green_heart: | mvnsite | 1m 52s | trunk passed | |
+1 :green_heart: | javadoc | 1m 38s | trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javadoc | 1m 2s | trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 2m 56s | trunk passed | |
+1 :green_heart: | shadedclient | 23m 28s | branch has no errors when building and testing our client artifacts. | |
_ Patch Compile Tests _ | ||||
+1 :green_heart: | mvninstall | 1m 1s | the patch passed | |
+1 :green_heart: | compile | 22m 43s | the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javac | 22m 43s | the patch passed | |
+1 :green_heart: | compile | 20m 54s | the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | javac | 20m 54s | the patch passed | |
+1 :green_heart: | blanks | 0m 0s | The patch has no blanks issues. | |
+1 :green_heart: | checkstyle | 1m 18s | hadoop-common-project/hadoop-common: The patch generated 0 new + 145 unchanged - 3 fixed = 145 total (was 148) | |
+1 :green_heart: | mvnsite | 2m 8s | the patch passed | |
+1 :green_heart: | javadoc | 1m 17s | the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 | |
+1 :green_heart: | javadoc | 1m 9s | the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 | |
+1 :green_heart: | spotbugs | 3m 11s | the patch passed | |
+1 :green_heart: | shadedclient | 23m 51s | patch has no errors when building and testing our client artifacts. | |
_ Other Tests _ | ||||
+1 :green_heart: | unit | 18m 48s | hadoop-common in the patch passed. | |
+1 :green_heart: | asflicense | 1m 24s | The patch does not generate ASF License warnings. | |
215m 24s |
Subsystem | Report/Notes |
---|---|
Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/8/artifact/out/Dockerfile |
GITHUB PR | https://github.com/apache/hadoop/pull/4527 |
Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets |
uname | Linux 7a2eb1381b47 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
Build tool | maven |
Personality | dev-support/bin/hadoop.sh |
git revision | trunk / d4967eba40fff91eb5233a6b7cf5b03aca5edea7 |
Default Java | Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 |
Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 |
Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/8/testReport/ |
Max. process+thread count | 3159 (vs. ulimit of 5500) |
modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4527/8/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.