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-24211: Create table is slow in large cluster when AccessController is enabled. #1546
Conversation
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
@@ -741,6 +741,10 @@ public static int getNumberOfChildren(ZKWatcher zkw, String znode) | |||
if (nodes != null) { | |||
List<NodeAndData> newNodes = new ArrayList<>(); | |||
for (String node : nodes) { | |||
if (Thread.interrupted()) { | |||
// Partial data should not be processed. Cancel processing by sending empty list. | |||
return new ArrayList<>(); |
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.
returning Collections.emptyList()
would be better
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.
done
ZKUtil.getChildDataAndWatchForNewChildren(watcher, aclZNode); | ||
refreshNodes(nodeList); | ||
} catch (KeeperException ke) { | ||
LOG.error("Error reading data from zookeeper for path " + path, ke); |
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.
Please user {}
for LOG param path
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.
Or better create a string variable and use it for both LOG.error and abort message.
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.
Done
hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
Show resolved
Hide resolved
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
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
hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
Show resolved
Hide resolved
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
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, please raise PR for other branches.
@arshadmohammad Please let me know once you create PRs for branch-2 and branch-1. We can target to merge this to master, branch-2, 2.3 (to be covered by branch-2) and branch-1. |
This PR code applies to branch-2 branch-2.3. |
…ler is enabled. (#1546) Signed-off-by: Viraj Jasani <vjasani@apache.org> Signed-off-by: Pankaj <pankajkumar@apache.org>
…ler is enabled. (#1546) Signed-off-by: Viraj Jasani <vjasani@apache.org> Signed-off-by: Pankaj <pankajkumar@apache.org>
This PR changes were reverted back. Created new PR #1631 |
Create table is slow in large cluster when AccessController is enabled.