svn commit: r40861 - /release/incubator/pinot/KEYS

2020-08-04 Thread tingchen
Author: tingchen Date: Wed Aug 5 05:31:02 2020 New Revision: 40861 Log: Update Pinot KEYS file for 1AAD3FDA Modified: release/incubator/pinot/KEYS Modified: release/incubator/pinot/KEYS == ---

[GitHub] [incubator-pinot] Jackie-Jiang commented on pull request #5805: Pre-generate aggregation functions in QueryContext

2020-08-04 Thread GitBox
Jackie-Jiang commented on pull request #5805: URL: https://github.com/apache/incubator-pinot/pull/5805#issuecomment-668983180 @mayankshriv There is no way to ensure new implementations are stateless (we cannot prevent bugs in new implementation). The existing ones are covered by tests.

[GitHub] [incubator-pinot] suvodeep-pyne opened a new pull request #5808: [TE] Dashboard Resource Refactor

2020-08-04 Thread GitBox
suvodeep-pyne opened a new pull request #5808: URL: https://github.com/apache/incubator-pinot/pull/5808 This change refactors the resource classes. - Created a RootResource to hold the "/" namespace - Most dependencies have been moved to this resource simplifying the

[incubator-pinot] branch support_aggregation_function_name_with_underscore updated (a8b1912 -> 8912099)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch support_aggregation_function_name_with_underscore in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard a8b1912 update codecov version to v1.0.12 discard 51a27b7

[incubator-pinot] branch master updated (17a3873 -> 43b01ef)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 17a3873 Improve performance of DistinctCountThetaSketch by eliminating empty sketches and unions.

[GitHub] [incubator-pinot] fx19880617 merged pull request #5806: Fixing codecov

2020-08-04 Thread GitBox
fx19880617 merged pull request #5806: URL: https://github.com/apache/incubator-pinot/pull/5806 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5806: Fixing codecov

2020-08-04 Thread GitBox
fx19880617 commented on a change in pull request #5806: URL: https://github.com/apache/incubator-pinot/pull/5806#discussion_r465398911 ## File path: .github/workflows/pinot_tests.yml ## @@ -37,11 +37,8 @@ jobs: RUN_INTEGRATION_TESTS: false run:

[incubator-pinot] branch fixing_codecov created (now 92caa10)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch fixing_codecov in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 92caa10 Fixing codecov No new revisions were added by this update.

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5795: Support aggregation function name with underscore inside

2020-08-04 Thread GitBox
Jackie-Jiang commented on a change in pull request #5795: URL: https://github.com/apache/incubator-pinot/pull/5795#discussion_r465386143 ## File path: pinot-common/src/main/java/org/apache/pinot/common/function/AggregationFunctionType.java ## @@ -101,6 +101,13 @@ public

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5795: Support aggregation function name with underscore inside

2020-08-04 Thread GitBox
Jackie-Jiang commented on a change in pull request #5795: URL: https://github.com/apache/incubator-pinot/pull/5795#discussion_r465385004 ## File path: pinot-common/src/main/java/org/apache/pinot/common/function/AggregationFunctionType.java ## @@ -101,6 +101,13 @@ public

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5806: Fixing codecov

2020-08-04 Thread GitBox
Jackie-Jiang commented on a change in pull request #5806: URL: https://github.com/apache/incubator-pinot/pull/5806#discussion_r465382908 ## File path: .github/workflows/pinot_tests.yml ## @@ -37,11 +37,8 @@ jobs: RUN_INTEGRATION_TESTS: false run:

[GitHub] [incubator-pinot] vincentchenjl commented on a change in pull request #5799: [TE] Fix issue of not loading RCA template when metricid is not specified

2020-08-04 Thread GitBox
vincentchenjl commented on a change in pull request #5799: URL: https://github.com/apache/incubator-pinot/pull/5799#discussion_r465371945 ## File path: thirdeye/thirdeye-frontend/app/pods/rootcause/route.js ## @@ -296,13 +312,16 @@ export default

[GitHub] [incubator-pinot] fx19880617 opened a new issue #5807: Proposal: Deprecate PQL

2020-08-04 Thread GitBox
fx19880617 opened a new issue #5807: URL: https://github.com/apache/incubator-pinot/issues/5807 Pinot community, Current Pinot supports two query syntaxes and endpoints: PQL and SQL: PQL is the legacy implementation based on Antlr; SQL implementation is based on CalciteSQL. It

[incubator-pinot] branch fixing_codecov updated (9bdd5af -> 92caa10)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch fixing_codecov in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 9bdd5af Fixing codecov add 92caa10 Fixing codecov This update added new revisions after

[incubator-pinot] branch master updated: Improve performance of DistinctCountThetaSketch by eliminating empty sketches and unions. (#5798)

2020-08-04 Thread mayanks
This is an automated email from the ASF dual-hosted git repository. mayanks pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/master by this push: new 17a3873 Improve performance of

[GitHub] [incubator-pinot] mayankshriv merged pull request #5798: Improve performance of DistinctCountThetaSketch by eliminating empty …

2020-08-04 Thread GitBox
mayankshriv merged pull request #5798: URL: https://github.com/apache/incubator-pinot/pull/5798 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-pinot] mangrrua commented on a change in pull request #5787: [Feature] - Spark Pinot Connector

2020-08-04 Thread GitBox
mangrrua commented on a change in pull request #5787: URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464311850 ## File path: config/.scalafmt.conf ## @@ -0,0 +1,17 @@ +version = "2.4.0" Review comment: Defined rules to format scala code.

[GitHub] [incubator-pinot] siddharthteotia merged pull request #5796: Move request optimizers and parser factory to pinot-core

2020-08-04 Thread GitBox
siddharthteotia merged pull request #5796: URL: https://github.com/apache/incubator-pinot/pull/5796 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[incubator-pinot] branch master updated (5d66b9e -> f2a990b)

2020-08-04 Thread siddteotia
This is an automated email from the ASF dual-hosted git repository. siddteotia pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 5d66b9e [TE] The endpoint for searching anomalies and pagination (#5778) add f2a990b Move

[incubator-pinot] branch fixing_codecov updated (cfcadf0 -> 9bdd5af)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch fixing_codecov in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard cfcadf0 Fixing codecov add 9bdd5af Fixing codecov This update added new revisions after

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #5806: Fixing codecov

2020-08-04 Thread GitBox
fx19880617 opened a new pull request #5806: URL: https://github.com/apache/incubator-pinot/pull/5806 ## Description Fixing codecov by changing github action plugins to bash command. This is an automated message from

[incubator-pinot] 01/01: Fixing codecov

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch fixing_codecov in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit cfcadf06ed5283a71865314798f73e0ea1fef1f5 Author: Xiang Fu AuthorDate: Tue Aug 4 13:05:17 2020 -0700

[incubator-pinot] branch fixing_codecov created (now cfcadf0)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch fixing_codecov in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at cfcadf0 Fixing codecov This branch includes the following new commits: new cfcadf0 Fixing

[incubator-pinot] branch master updated (7cc75ad -> 5d66b9e)

2020-08-04 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 7cc75ad [TE] make rca insights in email resilient to issues in rca response (#5794) add 5d66b9e [TE]

[GitHub] [incubator-pinot] jihaozh merged pull request #5778: [TE] The endpoint for searching anomalies and pagination

2020-08-04 Thread GitBox
jihaozh merged pull request #5778: URL: https://github.com/apache/incubator-pinot/pull/5778 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #5798: Improve performance of DistinctCountThetaSketch by eliminating empty …

2020-08-04 Thread GitBox
mayankshriv commented on a change in pull request #5798: URL: https://github.com/apache/incubator-pinot/pull/5798#discussion_r465297357 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java ##

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #5805: Pre-generate aggregation functions in QueryContext

2020-08-04 Thread GitBox
Jackie-Jiang opened a new pull request #5805: URL: https://github.com/apache/incubator-pinot/pull/5805 ## Description `AggregationFunction` itself is stateless, so we can share it among all the segments to prevent the overhead of creating it per segment. This can significantly improve

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #5798: Improve performance of DistinctCountThetaSketch by eliminating empty …

2020-08-04 Thread GitBox
mayankshriv commented on a change in pull request #5798: URL: https://github.com/apache/incubator-pinot/pull/5798#discussion_r465294347 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java ##

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #5798: Improve performance of DistinctCountThetaSketch by eliminating empty …

2020-08-04 Thread GitBox
mayankshriv commented on a change in pull request #5798: URL: https://github.com/apache/incubator-pinot/pull/5798#discussion_r465294347 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java ##

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #5798: Improve performance of DistinctCountThetaSketch by eliminating empty …

2020-08-04 Thread GitBox
mayankshriv commented on a change in pull request #5798: URL: https://github.com/apache/incubator-pinot/pull/5798#discussion_r465294347 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java ##

[GitHub] [incubator-pinot] akshayrai merged pull request #5794: [TE] make rca insights in email resilient to issues in rca response

2020-08-04 Thread GitBox
akshayrai merged pull request #5794: URL: https://github.com/apache/incubator-pinot/pull/5794 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[incubator-pinot] branch master updated (999f567 -> 7cc75ad)

2020-08-04 Thread akshayrai09
This is an automated email from the ASF dual-hosted git repository. akshayrai09 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 999f567 [TE] Guice Injection for Dashboard Server (#5792) add 7cc75ad [TE] make rca insights in

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5794: [TE] make rca insights in email resilient to issues in rca response

2020-08-04 Thread GitBox
akshayrai commented on a change in pull request #5794: URL: https://github.com/apache/incubator-pinot/pull/5794#discussion_r465275426 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/SummaryResource.java ## @@ -172,6 +158,57 @@

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #5778: [TE] The endpoint for searching anomalies and pagination

2020-08-04 Thread GitBox
jihaozh commented on a change in pull request #5778: URL: https://github.com/apache/incubator-pinot/pull/5778#discussion_r465261202 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/v2/anomalies/AnomalySearchResource.java ## @@

[GitHub] [incubator-pinot] vincentchenjl commented on a change in pull request #5794: [TE] make rca insights in email resilient to issues in rca response

2020-08-04 Thread GitBox
vincentchenjl commented on a change in pull request #5794: URL: https://github.com/apache/incubator-pinot/pull/5794#discussion_r465258807 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/SummaryResource.java ## @@ -172,6

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #5778: [TE] The endpoint for searching anomalies and pagination

2020-08-04 Thread GitBox
jihaozh commented on a change in pull request #5778: URL: https://github.com/apache/incubator-pinot/pull/5778#discussion_r465255934 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/v2/anomalies/AnomalySearcher.java ## @@ -0,0

[incubator-pinot] branch anomalies-pagination updated (9586523 -> d411dcd)

2020-08-04 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch anomalies-pagination in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 9586523 address comments discard ee879b1 - new anomalies endpoint and unit tests add

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5778: [TE] The endpoint for searching anomalies and pagination

2020-08-04 Thread GitBox
akshayrai commented on a change in pull request #5778: URL: https://github.com/apache/incubator-pinot/pull/5778#discussion_r465204182 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/v2/anomalies/AnomalySearchResource.java ##

[incubator-pinot] branch master updated (d1b4586 -> 999f567)

2020-08-04 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from d1b4586 New endpoint to get routing table for sql query (#5791) add 999f567 [TE] Guice Injection for

[GitHub] [incubator-pinot] jihaozh merged pull request #5792: [TE] Refactor Dashboard Server resources and dependencies

2020-08-04 Thread GitBox
jihaozh merged pull request #5792: URL: https://github.com/apache/incubator-pinot/pull/5792 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-pinot] lgo opened a new issue #5804: File processing errors during `CreateSegment` does not print the name of the bad file

2020-08-04 Thread GitBox
lgo opened a new issue #5804: URL: https://github.com/apache/incubator-pinot/issues/5804 When running the `CreateSegment` tool from the launcher script, I'd accidentally provided it a bad JSON file because the file was truncated (I believe a string wasn't closed). When the error

[GitHub] [incubator-pinot] lgo opened a new issue #5803: Controllers do not have a `/health` resource

2020-08-04 Thread GitBox
lgo opened a new issue #5803: URL: https://github.com/apache/incubator-pinot/issues/5803 Controllers do not have a healthcheck resource at `/health`, which is inconsistent with servers and brokers which do have an explicit `/health` resource for doing healthchecks. It'd be nice for all

[GitHub] [incubator-pinot] lgo opened a new issue #5802: Running Pinot with an invalid zookeeper service name seems to error during clean-up (in addition to connection error)

2020-08-04 Thread GitBox
lgo opened a new issue #5802: URL: https://github.com/apache/incubator-pinot/issues/5802 Running with an invalid zookeeper service name seems to have an error during clean-up in addition to the main error, eg: `bin/pinot-admin.sh StartController -clusterName PinotCluster -controllerHost

[GitHub] [incubator-pinot] kishoreg commented on issue #5801: Configuation for nodes (zookeeper & config files) is confusion and inconsistent between node types

2020-08-04 Thread GitBox
kishoreg commented on issue #5801: URL: https://github.com/apache/incubator-pinot/issues/5801#issuecomment-668725705 related issue https://github.com/apache/incubator-pinot/issues/5580 This is an automated message from the

[GitHub] [incubator-pinot] lgo opened a new issue #5801: Configuation for nodes is confusion and inconsistent between node types

2020-08-04 Thread GitBox
lgo opened a new issue #5801: URL: https://github.com/apache/incubator-pinot/issues/5801 Configuing nodes is faily confusing! For example, brokers and servers require `-zkAddress` parameters, whether or not they are provided a `-configFileName`. Meanwhile, controllers do not allow that

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5798: Improve performance of DistinctCountThetaSketch by eliminating empty …

2020-08-04 Thread GitBox
siddharthteotia commented on a change in pull request #5798: URL: https://github.com/apache/incubator-pinot/pull/5798#discussion_r465201499 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java

[GitHub] [incubator-pinot] harleyjj commented on a change in pull request #5799: [TE] Fix issue of not loading RCA template when metricid is not specified

2020-08-04 Thread GitBox
harleyjj commented on a change in pull request #5799: URL: https://github.com/apache/incubator-pinot/pull/5799#discussion_r465191989 ## File path: thirdeye/thirdeye-frontend/app/pods/rootcause/route.js ## @@ -296,13 +312,16 @@ export default

[GitHub] [incubator-pinot] kishoreg commented on issue #5800: Problem to understand the Thrideye into Pinot

2020-08-04 Thread GitBox
kishoreg commented on issue #5800: URL: https://github.com/apache/incubator-pinot/issues/5800#issuecomment-668646422 @suvodeep-pyne ^^ This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] MarceloNunesAlves opened a new issue #5800: Problem to understand the Thrideye into Pinot

2020-08-04 Thread GitBox
MarceloNunesAlves opened a new issue #5800: URL: https://github.com/apache/incubator-pinot/issues/5800 What should I do to The Thrideye works in the Pinot? I didn't find anything about how it use this into the Pinot. Is there a documentation? I executed  Thrideye only in the Eclipse but I

[incubator-pinot] branch support_aggregation_function_name_with_underscore updated (085a95c -> a8b1912)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch support_aggregation_function_name_with_underscore in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 085a95c update codecov version to v1.0.12 add a8b1912

[incubator-pinot] branch support_aggregation_function_name_with_underscore updated (c1f5ec7 -> 085a95c)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch support_aggregation_function_name_with_underscore in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard c1f5ec7 update codecov version to v1.0.12 add 085a95c

[incubator-pinot] branch support_aggregation_function_name_with_underscore updated (6c5ca9d -> c1f5ec7)

2020-08-04 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch support_aggregation_function_name_with_underscore in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 6c5ca9d update codecov version to v1.0.5 add c1f5ec7