Skip to content

Conversation

@bharatviswa504
Copy link
Contributor

No description provided.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 48 Docker mode activated.
_ Prechecks _
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 1 new or modified test files.
_ trunk Compile Tests _
+1 mvninstall 1173 trunk passed
-1 compile 51 ozone-manager in trunk failed.
+1 checkstyle 20 trunk passed
-1 mvnsite 24 ozone-manager in trunk failed.
+1 shadedclient 721 branch has no errors when building and testing our client artifacts.
-1 findbugs 23 ozone-manager in trunk failed.
+1 javadoc 21 trunk passed
_ Patch Compile Tests _
-1 mvninstall 19 ozone-manager in the patch failed.
-1 compile 17 ozone-manager in the patch failed.
-1 javac 17 ozone-manager in the patch failed.
+1 checkstyle 13 the patch passed
-1 mvnsite 19 ozone-manager in the patch failed.
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 773 patch has no errors when building and testing our client artifacts.
-1 findbugs 21 ozone-manager in the patch failed.
+1 javadoc 19 the patch passed
_ Other Tests _
-1 unit 21 ozone-manager in the patch failed.
+1 asflicense 27 The patch does not generate ASF License warnings.
3104
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/Dockerfile
GITHUB PR #623
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 6b1286accc19 3.13.0-153-generic #203-Ubuntu SMP Thu Jun 14 08:52:28 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 568d3ab
maven version: Apache Maven 3.3.9
Default Java 1.8.0_191
compile https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/branch-compile-hadoop-ozone_ozone-manager.txt
mvnsite https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/branch-mvnsite-hadoop-ozone_ozone-manager.txt
findbugs https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/branch-findbugs-hadoop-ozone_ozone-manager.txt
mvninstall https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/patch-mvninstall-hadoop-ozone_ozone-manager.txt
compile https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/patch-compile-hadoop-ozone_ozone-manager.txt
javac https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/patch-compile-hadoop-ozone_ozone-manager.txt
mvnsite https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/patch-mvnsite-hadoop-ozone_ozone-manager.txt
findbugs https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/patch-findbugs-hadoop-ozone_ozone-manager.txt
unit https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/artifact/out/patch-unit-hadoop-ozone_ozone-manager.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/testReport/
Max. process+thread count 306 (vs. ulimit of 5500)
modules C: hadoop-ozone/ozone-manager U: hadoop-ozone/ozone-manager
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-623/1/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@arp7
Copy link
Contributor

arp7 commented Mar 19, 2019

+1 this is a comment-only change. Let's get it in so we can start seeing clean RAT reports from Jenkins.

@arp7 arp7 merged commit 310ebf5 into apache:trunk Mar 19, 2019
@bharatviswa504 bharatviswa504 deleted the HDDS-1308 branch March 19, 2019 21:10
shanthoosh pushed a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
prateekm vjagadish1989 whenever you get a chance. This is an initial set of mostly minor changes.

Author: Ahmed Abdul Hamid <[email protected]>

Reviewers: Jagadish<[email protected]>

Closes apache#623 from ahmedahamid/dev/ahabdulh/execution-planner-stylistic-improv
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