Skip to content

Conversation

@GuoPhilipse
Copy link
Member

image
image

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 57s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 yamllint 0m 0s yamllint was not available.
+0 🆗 markdownlint 0m 0s markdownlint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ trunk Compile Tests _
+0 🆗 mvndep 12m 27s Maven dependency ordering for branch
+1 💚 mvninstall 26m 49s trunk passed
+1 💚 mvnsite 34m 2s trunk passed
+1 💚 shadedclient 96m 22s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 26s Maven dependency ordering for patch
+1 💚 mvninstall 25m 43s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 mvnsite 21m 45s the patch passed
+1 💚 shadedclient 44m 17s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 0m 38s The patch does not generate ASF License warnings.
169m 51s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3967/1/artifact/out/Dockerfile
GITHUB PR #3967
Optional Tests dupname asflicense codespell yamllint mvnsite markdownlint
uname Linux 3a54dcc56a67 4.15.0-162-generic #170-Ubuntu SMP Mon Oct 18 11:38:05 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / c801493
Max. process+thread count 575 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common . U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3967/1/console
versions git=2.25.1 maven=3.6.3
Powered by Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Copy link
Member

@aajisaka aajisaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@aajisaka aajisaka merged commit b689643 into apache:trunk Feb 13, 2022
@aajisaka
Copy link
Member

Merged. Thank you @GuoPhilipse and @ayushtkn

aajisaka pushed a commit that referenced this pull request Feb 13, 2022
…able (#3967)

Reviewed-by: Ayush Saxena <[email protected]>
Signed-off-by: Akira Ajisaka <[email protected]>
(cherry picked from commit b689643)
aajisaka pushed a commit that referenced this pull request Feb 13, 2022
…able (#3967)

Reviewed-by: Ayush Saxena <[email protected]>
Signed-off-by: Akira Ajisaka <[email protected]>
(cherry picked from commit b689643)
HarshitGupta11 pushed a commit to HarshitGupta11/hadoop that referenced this pull request Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants