[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access

2020-01-21 Thread GitBox
jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r369154305 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuota

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access

2020-01-20 Thread GitBox
jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r368773524 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuota

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access

2020-01-20 Thread GitBox
jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r368771972 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuota

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access

2020-01-16 Thread GitBox
jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r367724221 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuota

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access

2020-01-16 Thread GitBox
jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r367724281 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuota

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access

2020-01-16 Thread GitBox
jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r367724329 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuota

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access

2020-01-16 Thread GitBox
jackjlli commented on a change in pull request #4979: Update quota manager to reduce zk access URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r367724168 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/QueryQuotaConfig.java #