[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255388826 ## File path: pinot-common/src/main/java/com/linkedin/pinot/common/utils/CommonConstants.java

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255388826 ## File path: pinot-common/src/main/java/com/linkedin/pinot/common/utils/CommonConstants.java

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255388826 ## File path: pinot-common/src/main/java/com/linkedin/pinot/common/utils/CommonConstants.java

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255387783 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerReque

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255385156 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerReque

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255387757 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerReque

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255385156 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerReque

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255387643 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/RequestParams.j

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255386197 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerReque

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r255385156 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerReque

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r254907522 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/BaseBrokerReq

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r251149989 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/BaseBrokerReq

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r238044599 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/BaseBrokerReq

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r238044491 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/BaseBrokerReq

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r238044473 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/TableSchemaCa

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r238044386 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/BaseBrokerReq

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r23807 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/TableSchemaCa

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r237759521 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/TableSchemaCa

[GitHub] jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker

jackjlli commented on a change in pull request #3568: Add guava cache to cache table schema in pinot broker URL: https://github.com/apache/incubator-pinot/pull/3568#discussion_r237756832 ## File path: pinot-broker/src/main/java/com/linkedin/pinot/broker/requesthandler/TableSchemaCa