-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-23781 Removed deprecated createTableDescriptor(String) from HBaseTestingUtility #1146
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
Conversation
|
💔 -1 overall
This message was automatically generated. |
virajjasani
left a comment
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.
Test failures are relevant? Also, shell might be using this constructor so it’s worth running shell tests locally
|
@virajjasani Checked the shell usage and it doesn't seem to be used there. Also ran the shell tests locally. They are all passing. I merged the latest master. Let's wait for QA. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
virajjasani
left a comment
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.
+1 if latest test failures are flaky
ndimiduk
left a comment
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.
Thanks for taking on these cleanup tasks @HorizonNet .
|
💔 -1 overall
This message was automatically generated. |
|
Test failures seem to be different with each build. Let me dig into it. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
Fixed the remaining Checkstyle violation. The test failing in the last build passes locally. |
|
💔 -1 overall
This message was automatically generated. |
|
Both test failures seem to be unrelated to the change. Ran them several times locally. |
|
@ndimiduk @virajjasani Does your +1 stay? |
saintstack
left a comment
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.
LGTM
If failing tests did not use this method, go for the commit... else do a test rerun before committing.
virajjasani
left a comment
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.
+1 stays :)
No description provided.