Skip to content

HBASE-30145 Upgrade hbase-server to use junit5 Part17#8199

Open
liuxiaocs7 wants to merge 1 commit intoapache:masterfrom
liuxiaocs7:HBASE-30145
Open

HBASE-30145 Upgrade hbase-server to use junit5 Part17#8199
liuxiaocs7 wants to merge 1 commit intoapache:masterfrom
liuxiaocs7:HBASE-30145

Conversation

@liuxiaocs7
Copy link
Copy Markdown
Member

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 51 out of 51 changed files in this pull request and generated 5 comments.

try {
getRS().getRpcServices().closeRegion(null, crr);
Assert.assertTrue(false);
assertTrue(false);
assertGet(this.region, rowkey, fam1, qual1, value);
} catch (IOException e) {
assertTrue("Thread id " + threadNumber + " operation " + i + " failed.", false);
assertTrue(false, "Thread id " + threadNumber + " operation " + i + " failed.");
TEST_UTIL.shutdownMiniCluster();
}
assertTrue("Failed to validate against conflict hostname configurations", false);
assertTrue(false, "Failed to validate against conflict hostname configurations");
Comment on lines 204 to +206
private boolean ignoreNetworkInterface(NetworkInterface networkInterface) throws Exception {
return networkInterface == null || networkInterface.isLoopback() || networkInterface.isVirtual()
|| !networkInterface.isUp();
|| networkInterface.isPointToPoint() || !networkInterface.isUp();
Comment on lines +105 to +106
assertTrue(2 == workQueue.size(), "ScheduledExecutor#workQueue should equals 2, now is "
+ workQueue.size() + " please check region is close");
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.

2 participants