Skip to content

HBASE-22904 NPE occurs when RS send space quota usage report during HMaster init - #529

Merged
saintstack merged 2 commits into
apache:masterfrom
pankaj72981:HBASE-22904
Aug 25, 2019
Merged

HBASE-22904 NPE occurs when RS send space quota usage report during HMaster init#529
saintstack merged 2 commits into
apache:masterfrom
pankaj72981:HBASE-22904

Conversation

@pankaj72981

@pankaj72981pankaj72981 commented Aug 23, 2019

Copy link
Copy Markdown
Contributor

During HMaster failover, if RegionServer send space quota report to HMaster and MasterQuotaManager is not yet initialized then NPE will occur.

MasterQuotaManagerquotaManager = this.master.getMasterQuotaManager();

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec107Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall418master passed
+1compile80master passed
+1checkstyle109master passed
+1shadedjars375branch has no errors when building our shaded downstream artifacts.
+1javadoc52master passed
0spotbugs333Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs330master passed
_ Patch Compile Tests _
+1mvninstall427the patch passed
+1compile64the patch passed
+1javac64the patch passed
+1checkstyle93the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars344patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1188Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc44the patch passed
+1findbugs298the patch passed
_ Other Tests _
-1unit13508hbase-server in the patch failed.
+1asflicense25The patch does not generate ASF License warnings.
17977
SubsystemReport/Notes
DockerClient=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/1/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux ea33b424ba26 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / 767bb15
Default Java1.8.0_181
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/1/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/1/testReport/
Max. process+thread count4752 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/1/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@the-sakthithe-sakthi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the patch @pankaj72981 . Any UTs here?

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec41Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall356master passed
+1compile55master passed
+1checkstyle78master passed
+1shadedjars284branch has no errors when building our shaded downstream artifacts.
+1javadoc40master passed
0spotbugs259Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs255master passed
_ Patch Compile Tests _
+1mvninstall305the patch passed
+1compile58the patch passed
+1javac58the patch passed
+1checkstyle77the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars285patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck966Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc38the patch passed
+1findbugs286the patch passed
_ Other Tests _
+1unit9282hbase-server in the patch passed.
+1asflicense25The patch does not generate ASF License warnings.
12819
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/2/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 8b3be17f890d 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / 74fb204
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/2/testReport/
Max. process+thread count4804 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/2/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec211Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall332master passed
+1compile54master passed
+1checkstyle79master passed
+1shadedjars280branch has no errors when building our shaded downstream artifacts.
+1javadoc35master passed
0spotbugs255Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs253master passed
_ Patch Compile Tests _
+1mvninstall301the patch passed
+1compile56the patch passed
+1javac56the patch passed
+1checkstyle74the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars274patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck937Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc35the patch passed
+1findbugs254the patch passed
_ Other Tests _
-1unit18714hbase-server in the patch failed.
+1asflicense24The patch does not generate ASF License warnings.
22282
ReasonTests
Failed junit testshadoop.hbase.master.TestAssignmentManagerMetrics
hadoop.hbase.master.procedure.TestSCPWithReplicas
hadoop.hbase.replication.TestReplicationKillSlaveRS
hadoop.hbase.namespace.TestNamespaceAuditor
hadoop.hbase.client.TestSnapshotTemporaryDirectory
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas
hadoop.hbase.replication.TestReplicationSmallTestsSync
hadoop.hbase.tool.TestSecureBulkLoadHFiles
hadoop.hbase.client.TestCloneSnapshotFromClientNormal
hadoop.hbase.replication.TestReplicationSmallTests
hadoop.hbase.tool.TestBulkLoadHFiles
hadoop.hbase.replication.TestReplicationSyncUpTool
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/3/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 2faf256885d9 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / 2d45801
Default Java1.8.0_181
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/3/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/3/testReport/
Max. process+thread count5210 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/3/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec89Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall513master passed
+1compile75master passed
+1checkstyle109master passed
+1shadedjars397branch has no errors when building our shaded downstream artifacts.
+1javadoc45master passed
0spotbugs322Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs319master passed
_ Patch Compile Tests _
+1mvninstall396the patch passed
+1compile67the patch passed
+1javac67the patch passed
+1checkstyle88the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars370patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1299Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc42the patch passed
+1findbugs329the patch passed
_ Other Tests _
-1unit19412hbase-server in the patch failed.
+1asflicense48The patch does not generate ASF License warnings.
24088
ReasonTests
Failed junit testshadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated
hadoop.hbase.client.TestSnapshotTemporaryDirectory
hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas
hadoop.hbase.tool.TestSecureBulkLoadHFiles
hadoop.hbase.master.procedure.TestProcedurePriority
hadoop.hbase.client.TestAsyncTableAdminApi
hadoop.hbase.namespace.TestNamespaceAuditor
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/4/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 4d5665d78b01 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / 2d45801
Default Java1.8.0_181
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/4/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/4/testReport/
Max. process+thread count4697 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/4/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec37Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall343master passed
+1compile58master passed
+1checkstyle78master passed
+1shadedjars291branch has no errors when building our shaded downstream artifacts.
+1javadoc35master passed
0spotbugs246Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs244master passed
_ Patch Compile Tests _
+1mvninstall315the patch passed
+1compile56the patch passed
+1javac56the patch passed
+1checkstyle77the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars288patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck971Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc35the patch passed
+1findbugs257the patch passed
_ Other Tests _
+1unit8248hbase-server in the patch passed.
+1asflicense29The patch does not generate ASF License warnings.
11740
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/5/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 21387206c031 4.4.0-157-generic #185-Ubuntu SMP Tue Jul 23 09:17:01 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / 504fc52
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/5/testReport/
Max. process+thread count4758 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/5/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec41Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall319master passed
+1compile52master passed
+1checkstyle74master passed
+1shadedjars268branch has no errors when building our shaded downstream artifacts.
+1javadoc34master passed
0spotbugs260Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs257master passed
_ Patch Compile Tests _
+1mvninstall294the patch passed
+1compile51the patch passed
+1javac51the patch passed
+1checkstyle72the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars258patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck904Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc32the patch passed
+1findbugs264the patch passed
_ Other Tests _
+1unit8895hbase-server in the patch passed.
+1asflicense23The patch does not generate ASF License warnings.
12201
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/9/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux d84e8f8e8e00 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / ee9d986
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/9/testReport/
Max. process+thread count4622 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/9/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec140Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall544master passed
+1compile121master passed
+1checkstyle161master passed
+1shadedjars501branch has no errors when building our shaded downstream artifacts.
+1javadoc93master passed
0spotbugs350Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs348master passed
_ Patch Compile Tests _
+1mvninstall330the patch passed
+1compile57the patch passed
+1javac57the patch passed
+1checkstyle80the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars287patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1006Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc36the patch passed
+1findbugs271the patch passed
_ Other Tests _
-1unit17031hbase-server in the patch failed.
+1asflicense33The patch does not generate ASF License warnings.
21443
ReasonTests
Failed junit testshadoop.hbase.client.TestAsyncRegionAdminApi2
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas
hadoop.hbase.master.TestAssignmentManagerMetrics
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/7/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 7e64413762da 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 08:28:49 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / ee9d986
Default Java1.8.0_181
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/7/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/7/testReport/
Max. process+thread count4757 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/7/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec119Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author1The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall411master passed
+1compile55master passed
+1checkstyle90master passed
+1shadedjars292branch has no errors when building our shaded downstream artifacts.
+1javadoc36master passed
0spotbugs263Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs261master passed
_ Patch Compile Tests _
+1mvninstall313the patch passed
+1compile61the patch passed
+1javac61the patch passed
+1checkstyle90the patch passed
+1whitespace1The patch has no whitespace issues.
+1shadedjars305patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck984Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc35the patch passed
+1findbugs243the patch passed
_ Other Tests _
-1unit16606hbase-server in the patch failed.
+1asflicense32The patch does not generate ASF License warnings.
20325
ReasonTests
Failed junit testshadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas
hadoop.hbase.master.TestMasterShutdown
hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory
hadoop.hbase.master.TestAssignmentManagerMetrics
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/11/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 14b424541745 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 08:28:49 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / 94155fe
Default Java1.8.0_181
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/11/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/11/testReport/
Max. process+thread count4549 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/11/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@pankaj72981

Copy link
Copy Markdown
ContributorAuthor

Thanks for the patch @pankaj72981 . Any UTs here?

IMO UT is not required here, as we are throwing any exception and mocking is not useful.
Kindly let me know you opinion.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec39Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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.
_ master Compile Tests _
+1mvninstall415master passed
+1compile76master passed
+1checkstyle111master passed
+1shadedjars364branch has no errors when building our shaded downstream artifacts.
+1javadoc48master passed
0spotbugs332Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs329master passed
_ Patch Compile Tests _
+1mvninstall365the patch passed
+1compile76the patch passed
+1javac76the patch passed
+1checkstyle110the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars354patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1216Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc44the patch passed
+1findbugs350the patch passed
_ Other Tests _
+1unit9484hbase-server in the patch passed.
+1asflicense24The patch does not generate ASF License warnings.
13866
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/12/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 36f9841094c5 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-529/out/precommit/personality/provided.sh
git revisionmaster / 554b253
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/12/testReport/
Max. process+thread count4616 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-529/12/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@the-sakthithe-sakthi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@saintstack
saintstack merged commit 00581ac into apache:masterAug 25, 2019
asfgit pushed a commit that referenced this pull request Aug 25, 2019
…Master init (#529)
* NPE occurs when RS send space quota usage report during HMaster init
* Added the debug log
Signed-off-by: Sakthi <sakthi@apache.org>
Signed-off-by: stack <stack@apache.org>
asfgit pushed a commit that referenced this pull request Aug 25, 2019
…Master init (#529)
* NPE occurs when RS send space quota usage report during HMaster init
* Added the debug log
Signed-off-by: Sakthi <sakthi@apache.org>
Signed-off-by: stack <stack@apache.org>
asfgit pushed a commit that referenced this pull request Aug 25, 2019
…Master init (#529)
* NPE occurs when RS send space quota usage report during HMaster init
* Added the debug log
Signed-off-by: Sakthi <sakthi@apache.org>
Signed-off-by: stack <stack@apache.org>
infraio pushed a commit to infraio/hbase that referenced this pull request Aug 17, 2020
…Master init (apache#529)
* NPE occurs when RS send space quota usage report during HMaster init
* Added the debug log
Signed-off-by: Sakthi <sakthi@apache.org>
Signed-off-by: stack <stack@apache.org>
symat pushed a commit to symat/hbase that referenced this pull request Feb 17, 2021
…Master init (apache#529)
* NPE occurs when RS send space quota usage report during HMaster init
* Added the debug log
Signed-off-by: Sakthi <sakthi@apache.org>
Signed-off-by: stack <stack@apache.org>
(cherry picked from commit 481b2e3)
Change-Id: I932884bc65eda110370e737161da26cf4422744d
Sign up for freeto 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.

5 participants

@pankaj72981@Apache-HBase@busbey@the-sakthi@saintstack