Skip to content

Conversation

@shwetayakkali
Copy link

No description provided.

Change-Id: I94ac5ebb220fb7e46d55bccf499c3e4a8f84e677
Change-Id: I14b5a0c1683d0750130c9186c1b21bfdae531db3
…checkstyle in InnerNodeImpl

Change-Id: Ie9d3866e88b067c55a955fbb005a8c3460517338
Change-Id: I0470558d64a7a194c34ed79a0448c7a22f8c6f01
@shwetayakkali shwetayakkali deleted the HDDS-1651 branch June 7, 2019 23:09
@hanishakoneru
Copy link
Contributor

Hi @shwetayakkali, there seem to be some other commits also included in this PR. Can you please remove those.

@hanishakoneru hanishakoneru self-requested a review June 7, 2019 23:12
@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 47 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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 65 Maven dependency ordering for branch
+1 mvninstall 1626 trunk passed
+1 compile 1283 trunk passed
+1 checkstyle 255 trunk passed
+1 mvnsite 316 trunk passed
+1 shadedclient 1394 branch has no errors when building and testing our client artifacts.
+1 javadoc 259 trunk passed
0 spotbugs 108 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 576 trunk passed
_ Patch Compile Tests _
0 mvndep 28 Maven dependency ordering for patch
+1 mvninstall 258 the patch passed
+1 compile 1530 the patch passed
+1 javac 1530 the patch passed
+1 checkstyle 226 root: The patch generated 0 new + 230 unchanged - 15 fixed = 230 total (was 245)
+1 mvnsite 310 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 xml 1 The patch has no ill-formed XML file.
+1 shadedclient 763 patch has no errors when building and testing our client artifacts.
+1 javadoc 257 the patch passed
+1 findbugs 607 the patch passed
_ Other Tests _
+1 unit 537 hadoop-common in the patch passed.
+1 unit 131 hadoop-hdfs-client in the patch passed.
-1 unit 5731 hadoop-hdfs in the patch failed.
+1 unit 108 common in the patch passed.
+1 asflicense 71 The patch does not generate ASF License warnings.
16164
Reason Tests
Failed junit tests hadoop.hdfs.server.balancer.TestBalancer
hadoop.hdfs.server.datanode.TestDirectoryScanner
hadoop.hdfs.web.TestWebHdfsTimeouts
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-929/1/artifact/out/Dockerfile
GITHUB PR #929
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle xml
uname Linux 1e5cee6c89b1 4.4.0-141-generic #167~14.04.1-Ubuntu SMP Mon Dec 10 13:20:24 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 76b94c2
Default Java 1.8.0_212
unit https://builds.apache.org/job/hadoop-multibranch/job/PR-929/1/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-929/1/testReport/
Max. process+thread count 2739 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs hadoop-hdds/common U: .
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-929/1/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

shanthoosh pushed a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
…calaVersion

Currently we have scala suffix being interchangeably used for `scalaVersion` and `scalaLibVersion` for scala version.

Author: mynameborat <[email protected]>

Reviewers: Prateek Maheshwari <[email protected]>

Closes apache#929 from mynameborat/scala-cleanup
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.

3 participants