Skip to content

Conversation

@fuchaohong
Copy link
Contributor

@fuchaohong fuchaohong commented May 10, 2023

Description of PR

How was this patch tested?

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
  • Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
  • If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 35m 32s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shellcheck 0m 0s Shellcheck was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗 mvndep 15m 58s Maven dependency ordering for branch
+1 💚 mvninstall 20m 40s trunk passed
+1 💚 compile 14m 55s trunk passed
+1 💚 mvnsite 2m 15s trunk passed
+1 💚 javadoc 1m 37s trunk passed
+1 💚 shadedclient 20m 58s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 29s Maven dependency ordering for patch
+1 💚 mvninstall 1m 7s the patch passed
+1 💚 compile 13m 50s the patch passed
-1 ❌ cc 13m 50s /results-compile-cc-root.txt root generated 44 new + 144 unchanged - 0 fixed = 188 total (was 144)
+1 💚 golang 13m 50s the patch passed
+1 💚 javac 13m 50s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 mvnsite 2m 18s the patch passed
+1 💚 xmllint 0m 0s No new issues.
+1 💚 javadoc 1m 34s the patch passed
+1 💚 shadedclient 22m 16s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 0m 38s hadoop-project-dist in the patch passed.
-1 ❌ unit 18m 37s /patch-unit-hadoop-common-project_hadoop-common.txt hadoop-common in the patch passed.
+1 💚 asflicense 1m 4s The patch does not generate ASF License warnings.
174m 31s
Reason Tests
Failed junit tests hadoop.service.launcher.TestServiceInterruptHandling
hadoop.crypto.TestCryptoCodec
hadoop.io.nativeio.TestNativeIO
hadoop.crypto.TestCryptoStreamsWithOpensslSm4CtrCryptoCodec
Subsystem Report/Notes
Docker ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/1/artifact/out/Dockerfile
GITHUB PR #5635
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs compile javac javadoc mvninstall mvnsite unit shadedclient xmllint cc golang
uname Linux 306e7e622d16 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / fb29170
Default Java Red Hat, Inc.-1.8.0_362-b08
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/1/testReport/
Max. process+thread count 1251 (vs. ulimit of 5500)
modules C: hadoop-project-dist hadoop-common-project/hadoop-common U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/1/console
versions git=2.9.5 maven=3.6.3 xmllint=20901
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 35s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shellcheck 0m 0s Shellcheck was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗 mvndep 16m 10s Maven dependency ordering for branch
+1 💚 mvninstall 20m 14s trunk passed
+1 💚 compile 14m 32s trunk passed
+1 💚 mvnsite 2m 19s trunk passed
+1 💚 javadoc 1m 42s trunk passed
+1 💚 shadedclient 20m 49s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 29s Maven dependency ordering for patch
+1 💚 mvninstall 1m 6s the patch passed
+1 💚 compile 13m 44s the patch passed
+1 💚 cc 13m 44s the patch passed
+1 💚 golang 13m 44s the patch passed
+1 💚 javac 13m 44s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 mvnsite 2m 15s the patch passed
+1 💚 xmllint 0m 0s No new issues.
+1 💚 javadoc 1m 29s the patch passed
+1 💚 shadedclient 21m 48s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 0m 38s hadoop-project-dist in the patch passed.
-1 ❌ unit 18m 40s /patch-unit-hadoop-common-project_hadoop-common.txt hadoop-common in the patch passed.
+1 💚 asflicense 1m 5s The patch does not generate ASF License warnings.
138m 13s
Reason Tests
Failed junit tests hadoop.service.launcher.TestServiceInterruptHandling
hadoop.crypto.TestCryptoCodec
hadoop.crypto.TestCryptoStreamsWithOpensslSm4CtrCryptoCodec
Subsystem Report/Notes
Docker ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/2/artifact/out/Dockerfile
GITHUB PR #5635
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs compile javac javadoc mvninstall mvnsite unit shadedclient xmllint cc golang
uname Linux 082673a7cd2d 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / c3b14d7
Default Java Red Hat, Inc.-1.8.0_362-b08
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/2/testReport/
Max. process+thread count 2902 (vs. ulimit of 5500)
modules C: hadoop-project-dist hadoop-common-project/hadoop-common U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/2/console
versions git=2.9.5 maven=3.6.3 xmllint=20901
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 1m 11s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shellcheck 0m 0s Shellcheck was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗 mvndep 18m 14s Maven dependency ordering for branch
+1 💚 mvninstall 23m 3s trunk passed
+1 💚 compile 15m 48s trunk passed
+1 💚 mvnsite 2m 2s trunk passed
+1 💚 javadoc 1m 20s trunk passed
+1 💚 shadedclient 23m 43s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 24s Maven dependency ordering for patch
+1 💚 mvninstall 1m 5s the patch passed
+1 💚 compile 15m 3s the patch passed
+1 💚 cc 15m 3s the patch passed
+1 💚 golang 15m 3s the patch passed
+1 💚 javac 15m 3s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 mvnsite 2m 1s the patch passed
+1 💚 xmllint 0m 0s No new issues.
+1 💚 javadoc 1m 11s the patch passed
+1 💚 shadedclient 24m 37s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 0m 29s hadoop-project-dist in the patch passed.
-1 ❌ unit 18m 30s /patch-unit-hadoop-common-project_hadoop-common.txt hadoop-common in the patch passed.
+1 💚 asflicense 0m 54s The patch does not generate ASF License warnings.
151m 1s
Reason Tests
Failed junit tests hadoop.crypto.TestCryptoStreamsWithOpensslSm4CtrCryptoCodec
hadoop.crypto.TestCryptoCodec
hadoop.service.launcher.TestServiceInterruptHandling
Subsystem Report/Notes
Docker ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/3/artifact/out/Dockerfile
GITHUB PR #5635
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs compile javac javadoc mvninstall mvnsite unit shadedclient xmllint cc golang
uname Linux d7882882d152 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 7aff4bc
Default Java Red Hat, Inc.-1.8.0_362-b08
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/3/testReport/
Max. process+thread count 3134 (vs. ulimit of 5500)
modules C: hadoop-project-dist hadoop-common-project/hadoop-common U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/3/console
versions git=2.9.5 maven=3.6.3 xmllint=20901
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@fuchaohong
Copy link
Contributor Author

@Hexiaoqiao Can you please review my code, How does the reason for the check failure relate to my code.

@fuchaohong
Copy link
Contributor Author

@hadoop-yetus recheck

@jojochuang
Copy link
Contributor

Failed tests look unrelated. Triggered the rebuild to be sure.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 38s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shellcheck 0m 0s Shellcheck was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗 mvndep 22m 1s Maven dependency ordering for branch
+1 💚 mvninstall 20m 54s trunk passed
+1 💚 compile 15m 5s trunk passed
+1 💚 mvnsite 2m 14s trunk passed
+1 💚 javadoc 1m 38s trunk passed
+1 💚 shadedclient 21m 57s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 27s Maven dependency ordering for patch
+1 💚 mvninstall 1m 6s the patch passed
+1 💚 compile 14m 43s the patch passed
+1 💚 cc 14m 43s the patch passed
+1 💚 golang 14m 43s the patch passed
+1 💚 javac 14m 43s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 mvnsite 2m 4s the patch passed
+1 💚 xmllint 0m 0s No new issues.
+1 💚 javadoc 1m 15s the patch passed
+1 💚 shadedclient 22m 26s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 0m 31s hadoop-project-dist in the patch passed.
-1 ❌ unit 19m 14s /patch-unit-hadoop-common-project_hadoop-common.txt hadoop-common in the patch passed.
+1 💚 asflicense 1m 4s The patch does not generate ASF License warnings.
148m 12s
Reason Tests
Failed junit tests hadoop.service.launcher.TestServiceInterruptHandling
hadoop.crypto.TestCryptoCodec
hadoop.crypto.TestCryptoStreamsWithOpensslSm4CtrCryptoCodec
Subsystem Report/Notes
Docker ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/4/artifact/out/Dockerfile
GITHUB PR #5635
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs compile javac javadoc mvninstall mvnsite unit shadedclient xmllint cc golang
uname Linux 5e343ab42dbc 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 7aff4bc
Default Java Red Hat, Inc.-1.8.0_362-b08
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/4/testReport/
Max. process+thread count 2990 (vs. ulimit of 5500)
modules C: hadoop-project-dist hadoop-common-project/hadoop-common U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5635/4/console
versions git=2.9.5 maven=3.6.3 xmllint=20901
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@fuchaohong
Copy link
Contributor Author

The failure reason is still the same. @jojochuang do you have any comment?

@github-actions
Copy link
Contributor

We're closing this stale PR because it has been open for 100 days with no activity. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you feel like this was a mistake, or you would like to continue working on it, please feel free to re-open it and ask for a committer to remove the stale tag and review again.
Thanks all for your contribution.

@github-actions github-actions bot added the Stale label Oct 21, 2025
@github-actions github-actions bot closed this Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants