[GitHub] [incubator-pinot] vincentchenjl opened a new pull request #6036: [TE] fix labeler config mapping and timeout when fetching anomalies

vincentchenjl opened a new pull request #6036: URL: https://github.com/apache/incubator-pinot/pull/6036 This PR is to fix a few following issues. - ModelMapper cannot map severity configuration into Threshold objects. - Tunner fails when fetching anomalies gets timeout, which

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #6026: [TE] Creating a thirdeye-dashboard module to host the dashboard server

suvodeep-pyne commented on a change in pull request #6026: URL: https://github.com/apache/incubator-pinot/pull/6026#discussion_r491240900 ## File path: thirdeye/thirdeye-pinot/pom.xml ## @@ -361,52 +361,6 @@ - -maven-shade-plugin Review comment:

[GitHub] [incubator-pinot] sajjad-moradi opened a new pull request #6037: Add whitelist for emitting table level metrics

sajjad-moradi opened a new pull request #6037: URL: https://github.com/apache/incubator-pinot/pull/6037 ## Description Currently there is one config to enable/disable emitting table level metrics for all tables in a cluster. For big clusters, let's say the ones with a few thousands

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #6034: Publish helm package pinot 0.2.1

fx19880617 opened a new pull request #6034: URL: https://github.com/apache/incubator-pinot/pull/6034 ## Description Upgrade pinot helm to 0.2.1 This is an automated message from the Apache Git Service. To respond to the

[incubator-pinot] branch publish_helm_package_0.2.1 created (now 77def55)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch publish_helm_package_0.2.1 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 77def55 Publish helm package pinot 0.2.1 This branch includes the following new

[incubator-pinot] 01/01: Publish helm package pinot 0.2.1

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch publish_helm_package_0.2.1 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 77def552be3fc533428aba335fec2714fabdfcc2 Author: Xiang Fu AuthorDate: Fri Sep 18 13:48:34

[incubator-pinot] branch master updated: Publish helm package pinot 0.2.1 (#6034)

This is an automated email from the ASF dual-hosted git repository. xiangfu 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 8511410 Publish helm package pinot

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

Jackie-Jiang commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491193744 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/LLRealtimeSegmentDataManager.java ## @@ -1204,13

[GitHub] [incubator-pinot] Jackie-Jiang commented on pull request #6031: Handle the partitioning mismatch between table config and stream

Jackie-Jiang commented on pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#issuecomment-695098958 @mayankshriv @mcvsubbu Addressed all the comments, and minimize the change in this PR to just bring back the old behavior where we update the partition info when

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

Jackie-Jiang commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491201298 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java ## @@

[GitHub] [incubator-pinot] vincentchenjl commented on a change in pull request #6026: [TE] Creating a thirdeye-dashboard module to host the dashboard server

vincentchenjl commented on a change in pull request #6026: URL: https://github.com/apache/incubator-pinot/pull/6026#discussion_r491239194 ## File path: thirdeye/thirdeye-pinot/pom.xml ## @@ -361,52 +361,6 @@ - -maven-shade-plugin Review comment:

[GitHub] [incubator-pinot] Jackie-Jiang commented on pull request #6027: Support streaming query in QueryExecutor

Jackie-Jiang commented on pull request #6027: URL: https://github.com/apache/incubator-pinot/pull/6027#issuecomment-695141063 Discussed with @siddharthteotia and @fx19880617 offline and we all agreed that `QueryExecutor` logic for streaming and non-streaming query will remain very

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

Jackie-Jiang commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491194938 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/SegmentPartitionLLCRealtimeClusterIntegrationTest.java

[incubator-pinot] branch publish_helm_package_0.2.1 created (now 8934cf9)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch publish_helm_package_0.2.1 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 8934cf9 Publish helm repo with new index No new revisions were added by this update.

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

Jackie-Jiang commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491155277 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java ## @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

mcvsubbu commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491192210 ## File path: pinot-common/src/main/java/org/apache/pinot/common/metrics/ServerMeter.java ## @@ -39,6 +39,7 @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

mcvsubbu commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491192719 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java ## @@ -569,13

[GitHub] [incubator-pinot] fx19880617 merged pull request #6034: Publish helm package pinot 0.2.1

fx19880617 merged pull request #6034: URL: https://github.com/apache/incubator-pinot/pull/6034 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] Jackie-Jiang commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

Jackie-Jiang commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491191593 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java ## @@

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #6035: Publish helm repo with new index

fx19880617 opened a new pull request #6035: URL: https://github.com/apache/incubator-pinot/pull/6035 ## Description Publish helm repo with Pinot 0.2.1 index This is an automated message from the Apache Git Service. To

[incubator-pinot] branch master updated: Publish helm repo with new index (#6035)

This is an automated email from the ASF dual-hosted git repository. xiangfu 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 0dbe06d Publish helm repo with new

[GitHub] [incubator-pinot] fx19880617 merged pull request #6035: Publish helm repo with new index

fx19880617 merged pull request #6035: URL: https://github.com/apache/incubator-pinot/pull/6035 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] icefury71 opened a new pull request #6038: Adding dependency validation check on Indexing config

icefury71 opened a new pull request #6038: URL: https://github.com/apache/incubator-pinot/pull/6038 Related to #5942 ## Description This PR adds a check to ensure any columns specified in inverted index or bloom filter config doesn't exist in the no dictionary column config.

[GitHub] [incubator-pinot] mr-agrwal opened a new pull request #6033: Add Broker Reduce Time Log

mr-agrwal opened a new pull request #6033: URL: https://github.com/apache/incubator-pinot/pull/6033 ## Description Add log for Broker Reduce Time to capture more details of timing of query. This is an automated

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

mayankshriv commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491055817 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/SegmentPartitionLLCRealtimeClusterIntegrationTest.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #6031: Handle the partitioning mismatch between table config and stream

mcvsubbu commented on a change in pull request #6031: URL: https://github.com/apache/incubator-pinot/pull/6031#discussion_r491053143 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java ## @@ -510,6