Skip to content

HDDS-1180. TestRandomKeyGenerator fails with NPE - #519

Closed
elek wants to merge 1 commit into
apache:trunkfrom
elek:HDDS-1180
Closed

HDDS-1180. TestRandomKeyGenerator fails with NPE#519
elek wants to merge 1 commit into
apache:trunkfrom
elek:HDDS-1180

Conversation

@elek

@elekelek commented Feb 26, 2019

Copy link
Copy Markdown
Member

@elekelek added the ozone label Feb 26, 2019
@hadoop-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec28Docker mode activated.
_ Prechecks _
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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 _
+1mvninstall1006trunk passed
-1compile55tools in trunk failed.
+1checkstyle15trunk passed
-1mvnsite23tools in trunk failed.
+1shadedclient648branch has no errors when building and testing our client artifacts.
-1findbugs22tools in trunk failed.
+1javadoc21trunk passed
_ Patch Compile Tests _
-1mvninstall21tools in the patch failed.
-1compile19tools in the patch failed.
-1javac19tools in the patch failed.
+1checkstyle11the patch passed
-1mvnsite19tools in the patch failed.
+1whitespace0The patch has no whitespace issues.
+1shadedclient704patch has no errors when building and testing our client artifacts.
-1findbugs23tools in the patch failed.
+1javadoc19the patch passed
_ Other Tests _
-1unit24tools in the patch failed.
+1asflicense29The patch does not generate ASF License warnings.
2779
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/Dockerfile
GITHUB PR#519
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux 0009452f6cdc 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / 59ba355
mavenversion: Apache Maven 3.3.9
Default Java1.8.0_191
compilehttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/branch-compile-hadoop-ozone_tools.txt
mvnsitehttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/branch-mvnsite-hadoop-ozone_tools.txt
findbugshttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/branch-findbugs-hadoop-ozone_tools.txt
mvninstallhttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/patch-mvninstall-hadoop-ozone_tools.txt
compilehttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/patch-compile-hadoop-ozone_tools.txt
javachttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/patch-compile-hadoop-ozone_tools.txt
mvnsitehttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/patch-mvnsite-hadoop-ozone_tools.txt
findbugshttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/patch-findbugs-hadoop-ozone_tools.txt
unithttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/artifact/out/patch-unit-hadoop-ozone_tools.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/testReport/
Max. process+thread count411 (vs. ulimit of 5500)
modulesC: hadoop-ozone/tools U: hadoop-ozone/tools
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-519/1/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@elek

elek commented Feb 26, 2019

Copy link
Copy Markdown
MemberAuthor

See the ci/ozone/unit: Only one unit tests are failed: TestRandomKeyGenerator.ratisTest3 which is not a NPE any more.

@bharatviswa504

Copy link
Copy Markdown
Contributor

+1 LGTM.

@bharatviswa504

Copy link
Copy Markdown
Contributor

This has been already fixed and committed by HDDS-1174.

shanthoosh added a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
…rrupts.
**Problem:**
If a thread executing zkClient.close is interrupted, currently we swallow the ZkInterruptedException and proceed without closing the zookeeper connection.
This leads to ephemeral nodes of StreamProcessor lurking around in zookeeper after StreamProcessor shutdown.
Users had to wait till zookeeper server session timeout for the ephemeral nodes to get deleted.
**Change:**
Retry once on InterruptedException when closing the zkClient.
Misc changes:
* Remove unnecessary null checks.
* Remove unnecessary typecasts.
Author: Shanthoosh Venkataraman <santhoshvenkat1988@gmail.com>
Reviewers: Jagadish <jagadish@apache.org>
Closesapache#519 from shanthoosh/handle_interrupted_exception_in_zkclient_close
Sign up for freeto 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

@elek@hadoop-yetus@bharatviswa504