Skip to content

Conversation

@aryangupta1998
Copy link
Contributor

NOTICE

jQuery is upgraded from 3.3.1 to 3.5.1 at

hadoop/hadoop-tools/hadoop-sls/src/main/html/js/thirdparty

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 43m 2s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 jshint 0m 0s jshint 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 _
+1 💚 mvninstall 36m 0s trunk passed
+1 💚 compile 0m 31s trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1
+1 💚 compile 0m 29s trunk passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01
+1 💚 mvnsite 0m 34s trunk passed
+1 💚 shadedclient 55m 21s branch has no errors when building and testing our client artifacts.
+1 💚 javadoc 0m 28s trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1
+1 💚 javadoc 0m 28s trunk passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01
_ Patch Compile Tests _
+1 💚 mvninstall 0m 30s the patch passed
+1 💚 compile 0m 25s the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1
+1 💚 javac 0m 25s the patch passed
+1 💚 compile 0m 21s the patch passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01
+1 💚 javac 0m 21s the patch passed
+1 💚 mvnsite 0m 26s the patch passed
-1 ❌ whitespace 0m 0s /whitespace-tabs.txt The patch 910 line(s) with tabs.
+1 💚 shadedclient 17m 35s patch has no errors when building and testing our client artifacts.
+1 💚 javadoc 0m 24s the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1
+1 💚 javadoc 0m 22s the patch passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01
_ Other Tests _
+1 💚 unit 12m 3s hadoop-sls in the patch passed.
+1 💚 asflicense 0m 33s The patch does not generate ASF License warnings.
134m 59s
Subsystem Report/Notes
Docker ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2379/1/artifact/out/Dockerfile
GITHUB PR #2379
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient jshint
uname Linux 83d3916d81fb 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 / 69ef9b1
Default Java Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01
Multi-JDK versions /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2379/1/testReport/
Max. process+thread count 454 (vs. ulimit of 5500)
modules C: hadoop-tools/hadoop-sls U: hadoop-tools/hadoop-sls
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2379/1/console
versions git=2.17.1 maven=3.6.0
Powered by Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@mukul1987 mukul1987 left a comment

Choose a reason for hiding this comment

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

+1, the changes looks good to me.

@mukul1987 mukul1987 merged commit d60d5fe into apache:trunk Oct 19, 2020
asfgit pushed a commit that referenced this pull request May 13, 2021
Co-authored-by: Aryan Gupta
(cherry picked from commit d60d5fe)
jojochuang pushed a commit to jojochuang/hadoop that referenced this pull request May 23, 2023
Co-authored-by: Aryan Gupta
(cherry picked from commit d60d5fe)
Change-Id: I840fdeffe0a0b126b4268f59ee724449d1209fe0
(cherry picked from commit 9b087f7)
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