-
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-22879 user_permission command failed to show global permission #511
Conversation
💔 -1 overall
This message was automatically generated. |
5803246
to
01eeee5
Compare
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
@@ -35,7 +35,7 @@ | |||
@Test | |||
public void testRunShellTests() throws IOException { | |||
System.setProperty("shell.test.exclude", "replication_admin_test.rb,rsgroup_shell_test.rb," + | |||
"admin_test.rb,table_test.rb,quotas_test.rb"); | |||
"admin_test.rb,table_test.rb,quotas_test.rb,admin2_test.rb"); |
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.
What's the story of admin2_test.rb
, why is it related to this fix?
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's not related to this fix. But admin2_test.rb is already tested in TestAdminShell2, TestShell can skip it rather than test it twice?
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
💔 -1 overall
This message was automatically generated. |
…pache#511) (cherry picked from commit 947f6d4) Change-Id: If5de4d376b78a7a13ac26321a464dc91ae12f9de
No description provided.