-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HBASE-22551 TestMasterOperationsForRegionReplicas is flakey #295
Conversation
} finally { | ||
ADMIN.disableTable(tableName); | ||
ADMIN.deleteTable(tableName); | ||
} | ||
} | ||
|
||
@Test @Ignore("Enable when we have support for alter_table- HBASE-10361") | ||
@Test | ||
@Ignore("Enable when we have support for alter_table- HBASE-10361") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unrelated, but HBASE-10361 has merged ages ago. I will open a JIRA to remove @ignore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me include this in the new patch, just a one line change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh the test fails locally for me... Then let's use a new issue to address it...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It also fails for me and the try block catches the underlying exception from ADMIN.enableTable(tableName);
. Better to work on this on a new issue.
💔 -1 overall
This message was automatically generated. |
if (ADMIN != null) ADMIN.close(); | ||
if (CONNECTION != null && !CONNECTION.isClosed()) CONNECTION.close(); | ||
Closeables.close(ADMIN, true); | ||
Closeables.close(CONNECTION, true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a nice utility...
💔 -1 overall
This message was automatically generated. |
No description provided.