[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4397: Adding Support for Kafka 2.0 Consumer

2019-07-27 Thread GitBox
codecov-io edited a comment on issue #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#issuecomment-508060135 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4397?src=pr=h1) Report > :exclamation: No coverage uploaded

[incubator-pinot] branch kafka_2.0 updated (0260eb6 -> 1ad264d)

2019-07-27 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch kafka_2.0 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 0260eb6 Skip Kafka 2.0 tests if there is no kafka connectors related changes. add 1ad264d

[GitHub] [incubator-pinot] codecov-io commented on issue #4475: Add sql +, -, *, / symbol arith

2019-07-27 Thread GitBox
codecov-io commented on issue #4475: Add sql +, -, *, / symbol arith URL: https://github.com/apache/incubator-pinot/pull/4475#issuecomment-515731033 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4475?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] xueyumusic opened a new pull request #4475: Add sql +, -, *, / symbol arith

2019-07-27 Thread GitBox
xueyumusic opened a new pull request #4475: Add sql +, -, *, / symbol arith URL: https://github.com/apache/incubator-pinot/pull/4475 This PR adds sql +, -, *, / symbol arith based on recent new sql grammar support using calcite. Currently it look only support pql-like ADD, SUB, MULT, DIV

[GitHub] [incubator-pinot] jasperjiaguo opened a new pull request #4474: An auto recommendation for inverted index

2019-07-27 Thread GitBox
jasperjiaguo opened a new pull request #4474: An auto recommendation for inverted index URL: https://github.com/apache/incubator-pinot/pull/4474 A solution to issue https://github.com/apache/incubator-pinot/issues/4461 This

[GitHub] [incubator-pinot] jasperjiaguo commented on issue #4461: Proposal for Automated Inverted Index Recommendation

2019-07-27 Thread GitBox
jasperjiaguo commented on issue #4461: Proposal for Automated Inverted Index Recommendation URL: https://github.com/apache/incubator-pinot/issues/4461#issuecomment-515707342 https://cwiki.apache.org/confluence/display/PINOT/Automated+Inverted+Index+Recommendation+for+Pinot

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4472: Use hit counter to track max QPS per minute for broker

2019-07-27 Thread GitBox
mcvsubbu commented on a change in pull request #4472: Use hit counter to track max QPS per minute for broker URL: https://github.com/apache/incubator-pinot/pull/4472#discussion_r307974496 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HitCounter.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4472: Use hit counter to track max QPS per minute for broker

2019-07-27 Thread GitBox
mcvsubbu commented on a change in pull request #4472: Use hit counter to track max QPS per minute for broker URL: https://github.com/apache/incubator-pinot/pull/4472#discussion_r307974522 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HitCounter.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4472: Use hit counter to track max QPS per minute for broker

2019-07-27 Thread GitBox
mcvsubbu commented on a change in pull request #4472: Use hit counter to track max QPS per minute for broker URL: https://github.com/apache/incubator-pinot/pull/4472#discussion_r307974402 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HitCounter.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4472: Use hit counter to track max QPS per minute for broker

2019-07-27 Thread GitBox
mcvsubbu commented on a change in pull request #4472: Use hit counter to track max QPS per minute for broker URL: https://github.com/apache/incubator-pinot/pull/4472#discussion_r307974669 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HitCounter.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4473: Refactor HelixBrokerStarterTest

2019-07-27 Thread GitBox
mcvsubbu commented on a change in pull request #4473: Refactor HelixBrokerStarterTest URL: https://github.com/apache/incubator-pinot/pull/4473#discussion_r307974080 ## File path: pinot-core/src/test/java/org/apache/pinot/util/TestUtils.java ## @@ -159,6 +97,6 @@ public

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4473: Refactor HelixBrokerStarterTest

2019-07-27 Thread GitBox
mcvsubbu commented on a change in pull request #4473: Refactor HelixBrokerStarterTest URL: https://github.com/apache/incubator-pinot/pull/4473#discussion_r307973488 ## File path: pinot-broker/src/test/java/org/apache/pinot/broker/broker/HelixBrokerStarterTest.java ## @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4473: Refactor HelixBrokerStarterTest

2019-07-27 Thread GitBox
mcvsubbu commented on a change in pull request #4473: Refactor HelixBrokerStarterTest URL: https://github.com/apache/incubator-pinot/pull/4473#discussion_r307973422 ## File path: pinot-broker/src/test/java/org/apache/pinot/broker/broker/HelixBrokerStarterTest.java ## @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4473: Refactor HelixBrokerStarterTest

2019-07-27 Thread GitBox
mcvsubbu commented on a change in pull request #4473: Refactor HelixBrokerStarterTest URL: https://github.com/apache/incubator-pinot/pull/4473#discussion_r307973680 ## File path: pinot-broker/src/test/java/org/apache/pinot/broker/broker/HelixBrokerStarterTest.java ## @@