[incubator-pinot] 01/01: [maven-release-plugin] prepare release release-0.6.0-rc0

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 3bba45fc0c643dbabf05d9f022a63d32dbc66e9f Author: Jack Li(Analytics Engineering) AuthorDate: Fri Oct 23

[incubator-pinot] branch release-0.6.0-rc created (now 3bba45f)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 3bba45f [maven-release-plugin] prepare release release-0.6.0-rc0 This branch includes the

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
codecov-io edited a comment on pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#issuecomment-715567148 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6184?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
codecov-io edited a comment on pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#issuecomment-715567148 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6184?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6185: Relaxing timeColumnName validation

2020-10-23 Thread GitBox
codecov-io edited a comment on pull request #6185: URL: https://github.com/apache/incubator-pinot/pull/6185#issuecomment-715657660 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6185?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-io commented on pull request #6185: Relaxing timeColumnName validation

2020-10-23 Thread GitBox
codecov-io commented on pull request #6185: URL: https://github.com/apache/incubator-pinot/pull/6185#issuecomment-715657660 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6185?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] snleee opened a new pull request #6185: Relaxing timeColumnName validation

2020-10-23 Thread GitBox
snleee opened a new pull request #6185: URL: https://github.com/apache/incubator-pinot/pull/6185 Current validation code will fail when the time column name is set to empty string. We can relax the validation by treating the empty string the same as the null value. Added the unit

[GitHub] [incubator-pinot] jtao15 commented on a change in pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
jtao15 commented on a change in pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511235934 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartMinionCommand.java ## @@ -0,0 +1,110 @@ +/** + * Licensed to

[GitHub] [incubator-pinot] jtao15 commented on a change in pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
jtao15 commented on a change in pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511234621 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/ClusterTest.java ## @@ -184,7 +184,7 @@ protected void

[GitHub] [incubator-pinot] snleee commented on a change in pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
snleee commented on a change in pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511218564 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/ClusterTest.java ## @@ -184,7 +184,7 @@ protected void

[GitHub] [incubator-pinot] snleee commented on a change in pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
snleee commented on a change in pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511218200 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotMinion.java ## @@ -0,0 +1,37 @@ +/** + * Licensed to the Apache

[incubator-pinot] branch release-0.6.0-rc created (now cad3619)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at cad3619 [maven-release-plugin] prepare release release-0.6.0-rc1 This branch includes the

[incubator-pinot] 01/01: [maven-release-plugin] prepare release release-0.6.0-rc1

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit cad3619b98a524d26ae549a2c09566d32650f584 Author: Jack Li(Analytics Engineering) AuthorDate: Fri Oct 23

[incubator-pinot] 01/01: [maven-release-plugin] prepare release release-0.6.0-rc0

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit b3f1ff9b2a23a2a8fa44a76047b18eeaf5190367 Author: Jack Li(Analytics Engineering) AuthorDate: Fri Oct 23

[incubator-pinot] branch release-0.6.0-rc created (now b3f1ff9)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at b3f1ff9 [maven-release-plugin] prepare release release-0.6.0-rc0 This branch includes the

[GitHub] [incubator-pinot] jtao15 commented on a change in pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
jtao15 commented on a change in pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511187102 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotMinion.java ## @@ -0,0 +1,37 @@ +/** + * Licensed to the Apache

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6127: Rewrite possible array aggregation functions to one level

2020-10-23 Thread GitBox
codecov-io edited a comment on pull request #6127: URL: https://github.com/apache/incubator-pinot/pull/6127#issuecomment-706417804 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6127?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6127: Rewrite possible array aggregation functions to one level

2020-10-23 Thread GitBox
codecov-io edited a comment on pull request #6127: URL: https://github.com/apache/incubator-pinot/pull/6127#issuecomment-706417804 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6127?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
codecov-io edited a comment on pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#issuecomment-715567148 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6184?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-io commented on pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
codecov-io commented on pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#issuecomment-715567148 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6184?src=pr=h1) Report > Merging

[incubator-pinot] tag release-0.6.0-rc0 created (now bb6a3f4)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to tag release-0.6.0-rc0 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at bb6a3f4 (commit) No new revisions were added by this update.

[incubator-pinot] branch release-0.6.0-rc updated: [maven-release-plugin] prepare for next development iteration

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/release-0.6.0-rc by this push: new bb6a3f4

[incubator-pinot] branch rewrite_array_aggregation_functions updated (2f93b8b -> 08ee98f)

2020-10-23 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch rewrite_array_aggregation_functions in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 2f93b8b format discard 463d634 Rewrite possible array aggregation

[incubator-pinot] annotated tag release-0.7.0-rc0 updated (691b23c -> 4fa1f8c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to annotated tag release-0.7.0-rc0 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. *** WARNING: tag release-0.7.0-rc0 was modified! *** from 691b23c (commit) to 4fa1f8c

[incubator-pinot] branch release-0.6.0-rc created (now 691b23c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0 No new revisions were added by

[incubator-pinot] branch release-0.6.0-rc updated: [maven-release-plugin] prepare for next development iteration

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/release-0.6.0-rc by this push: new bb6a3f4

[incubator-pinot] annotated tag release-0.7.0-rc0 updated (691b23c -> 4fa1f8c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to annotated tag release-0.7.0-rc0 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. *** WARNING: tag release-0.7.0-rc0 was modified! *** from 691b23c (commit) to 4fa1f8c

[incubator-pinot] branch release-0.6.0-rc created (now 691b23c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0 No new revisions were added by

[incubator-pinot] branch rewrite_array_aggregation_functions updated (2f93b8b -> 08ee98f)

2020-10-23 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch rewrite_array_aggregation_functions in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 2f93b8b format discard 463d634 Rewrite possible array aggregation

[incubator-pinot] branch rewrite_array_aggregation_functions updated (2f93b8b -> 08ee98f)

2020-10-23 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch rewrite_array_aggregation_functions in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 2f93b8b format discard 463d634 Rewrite possible array aggregation

[GitHub] [incubator-pinot] fx19880617 edited a comment on pull request #6127: Rewrite possible array aggregation functions to one level

2020-10-23 Thread GitBox
fx19880617 edited a comment on pull request #6127: URL: https://github.com/apache/incubator-pinot/pull/6127#issuecomment-715083700 > High level question, should we overload `SUM`, `MIN`, `MAX` to directly work on MV columns? I feel it is much easier to use if these functions directly work

[incubator-pinot] annotated tag release-0.7.0-rc0 updated (691b23c -> 4fa1f8c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to annotated tag release-0.7.0-rc0 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. *** WARNING: tag release-0.7.0-rc0 was modified! *** from 691b23c (commit) to 4fa1f8c

[incubator-pinot] branch release-0.6.0-rc updated: [maven-release-plugin] prepare for next development iteration

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/release-0.6.0-rc by this push: new bb6a3f4

[incubator-pinot] branch release-0.6.0-rc created (now 691b23c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0 No new revisions were added by

[incubator-pinot] branch release-0.6.0-rc updated: [maven-release-plugin] prepare for next development iteration

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/release-0.6.0-rc by this push: new bb6a3f4

[incubator-pinot] annotated tag release-0.7.0-rc0 updated (691b23c -> 4fa1f8c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to annotated tag release-0.7.0-rc0 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. *** WARNING: tag release-0.7.0-rc0 was modified! *** from 691b23c (commit) to 4fa1f8c

[incubator-pinot] branch release-0.6.0-rc created (now 691b23c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0 No new revisions were added by

[incubator-pinot] annotated tag release-0.7.0-rc0 updated (691b23c -> 4fa1f8c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to annotated tag release-0.7.0-rc0 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. *** WARNING: tag release-0.7.0-rc0 was modified! *** from 691b23c (commit) to 4fa1f8c

[incubator-pinot] branch release-0.6.0-rc updated: [maven-release-plugin] prepare for next development iteration

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/release-0.6.0-rc by this push: new bb6a3f4

[incubator-pinot] branch release-0.6.0-rc created (now 691b23c)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0 No new revisions were added by

[incubator-pinot] branch master updated (73d2839 -> 5577e87)

2020-10-23 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 73d2839 Merge common APIs for Dictionary (#6176) add 5577e87 Add table level lock for segment

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #6165: Add table level lock for segment upload

2020-10-23 Thread GitBox
fx19880617 commented on a change in pull request #6165: URL: https://github.com/apache/incubator-pinot/pull/6165#discussion_r511102497 ## File path: pinot-spi/src/main/java/org/apache/pinot/spi/utils/retry/RandomDelayRetryPolicy.java ## @@ -0,0 +1,44 @@ +/** + * Licensed to

[GitHub] [incubator-pinot] fx19880617 merged pull request #6165: Add table level lock for segment upload

2020-10-23 Thread GitBox
fx19880617 merged pull request #6165: URL: https://github.com/apache/incubator-pinot/pull/6165 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] codecov-io edited a comment on pull request #6165: Add table level lock for segment upload

2020-10-23 Thread GitBox
codecov-io edited a comment on pull request #6165: URL: https://github.com/apache/incubator-pinot/pull/6165#issuecomment-713177186 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6165?src=pr=h1) Report > Merging

[incubator-pinot] branch te-remove-shade-plugin updated (b975140 -> 69c29c1)

2020-10-23 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch te-remove-shade-plugin in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from b975140 [TE] Remove maven-shade-plugin in thirdeye-dashboard (#6151) add 69c29c1 [TE]

[GitHub] [incubator-pinot] jihaozh merged pull request #6180: [TE] Log alert registry for debugging

2020-10-23 Thread GitBox
jihaozh merged pull request #6180: URL: https://github.com/apache/incubator-pinot/pull/6180 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

[incubator-pinot] branch add_table_lock_for_segment_upload updated (6d910f9 -> 27f84c4)

2020-10-23 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch add_table_lock_for_segment_upload in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 6d910f9 Using random delay policy add 27f84c4 update random policy No

[incubator-pinot] branch release-0.6.0-rc updated: [maven-release-plugin] prepare release release-0.6.0-rc0

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch release-0.6.0-rc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/release-0.6.0-rc by this push: new 7d91803

[GitHub] [incubator-pinot] mcvsubbu commented on issue #5346: Cleanup integration tests to get more coverage, less randomness

2020-10-23 Thread GitBox
mcvsubbu commented on issue #5346: URL: https://github.com/apache/incubator-pinot/issues/5346#issuecomment-715500233 > Vote for this solution: `One way to get this is to comb the 10k queries to select a few hundred that can be enough to detect all issues. And then we add queries to this

[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #5346: Cleanup integration tests to get more coverage, less randomness

2020-10-23 Thread GitBox
Jackie-Jiang commented on issue #5346: URL: https://github.com/apache/incubator-pinot/issues/5346#issuecomment-715497623 Vote for this solution: `One way to get this is to comb the 10k queries to select a few hundred that can be enough to detect all issues. And then we add queries to

[GitHub] [incubator-pinot] codecov-io commented on pull request #6181: Reduce pql queries from 10k to 500 in pinot-integration-tests module

2020-10-23 Thread GitBox
codecov-io commented on pull request #6181: URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-715496693 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6181?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] snleee commented on a change in pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
snleee commented on a change in pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511052394 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotMinion.java ## @@ -0,0 +1,37 @@ +/** + * Licensed to the Apache

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6181: Reduce pql queries from 10k to 500 in pinot-integration-tests module

2020-10-23 Thread GitBox
Jackie-Jiang commented on a change in pull request #6181: URL: https://github.com/apache/incubator-pinot/pull/6181#discussion_r511048205 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/BaseClusterIntegrationTestSet.java ## @@ -60,7 +60,7

[GitHub] [incubator-pinot] jtao15 opened a new pull request #6184: Minion command line starter for issue #6111

2020-10-23 Thread GitBox
jtao15 opened a new pull request #6184: URL: https://github.com/apache/incubator-pinot/pull/6184 Minion command line starter for issue #6111 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-pinot] mayankshriv commented on pull request #6181: Reduce pql queries from 10k to 500 in pinot-integration-tests module

2020-10-23 Thread GitBox
mayankshriv commented on pull request #6181: URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-715487591 @jackjlli Slightly tangential question, most users are either migrating to or already using SQL. Is there a SQL version these queries that we are validating

[incubator-pinot] branch refactor-integration-tests updated (d6d2c64 -> 6aa17fd)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch refactor-integration-tests in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard d6d2c64 Reduce pql queries from 10k to 500 add 6aa17fd Reduce pql queries from 10k

[GitHub] [incubator-pinot] icefury71 commented on pull request #6172: Adding a new Server API for computing average off heap memory consumed

2020-10-23 Thread GitBox
icefury71 commented on pull request #6172: URL: https://github.com/apache/incubator-pinot/pull/6172#issuecomment-715459444 > Should we also add api's to get off-heap memory consumption for: > > * All tables hosted on the server (multi-tenant case). > * Usage for a table across all

[GitHub] [incubator-pinot] jackjlli commented on pull request #6181: Reduce pql queries from 10k to 500 in pinot-integration-tests module

2020-10-23 Thread GitBox
jackjlli commented on pull request #6181: URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-715457398 > You may want to add a couple of queries that return empty results also. Not necessarily, the queries with empty response will skip the comparison of the

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #6172: Adding a new Server API for computing average off heap memory consumed

2020-10-23 Thread GitBox
mayankshriv commented on a change in pull request #6172: URL: https://github.com/apache/incubator-pinot/pull/6172#discussion_r511009756 ## File path: pinot-core/src/main/java/org/apache/pinot/core/realtime/impl/RealtimeSegmentStatsHistory.java ## @@ -385,7 +395,7 @@ public

[GitHub] [incubator-pinot] mayankshriv commented on pull request #6181: Reduce pql queries from 10k to 500 in pinot-integration-tests module

2020-10-23 Thread GitBox
mayankshriv commented on pull request #6181: URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-715452220 Do we have a coverage report for this one? Granted that queries with empty results are not effective, but if the code path is still exercised and we see a drop in

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #6165: Add table level lock for segment upload

2020-10-23 Thread GitBox
mcvsubbu commented on a change in pull request #6165: URL: https://github.com/apache/incubator-pinot/pull/6165#discussion_r510996450 ## File path: pinot-spi/src/main/java/org/apache/pinot/spi/utils/retry/RandomDelayRetryPolicy.java ## @@ -0,0 +1,44 @@ +/** + * Licensed to the

[GitHub] [incubator-pinot] sajjad-moradi commented on a change in pull request #6181: Reduce pql queries from 10k to 500 in pinot-integration-tests module

2020-10-23 Thread GitBox
sajjad-moradi commented on a change in pull request #6181: URL: https://github.com/apache/incubator-pinot/pull/6181#discussion_r510979784 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/BaseClusterIntegrationTestSet.java ## @@ -320,19

[GitHub] [incubator-pinot] cyrilou242 opened a new pull request #6183: [TE] allow `@` character in jira user name

2020-10-23 Thread GitBox
cyrilou242 opened a new pull request #6183: URL: https://github.com/apache/incubator-pinot/pull/6183 ## Description For the Jira Alert in ThirdEye: If a jira reporter user name contains a `@` character, eg `n...@mycompany.com`, an error is raised when trying to run the JQL

[GitHub] [incubator-pinot] venkatvghub opened a new issue #6182: [Thirdeye] - Anamoly images periodically fails to load.

2020-10-23 Thread GitBox
venkatvghub opened a new issue #6182: URL: https://github.com/apache/incubator-pinot/issues/6182 Hello, am seeing an image loading issue on the "Anamolies" page. At times, the images show up and at times it doesnt. Have to hit page load many a times to see a missing image. Same image

[GitHub] [incubator-pinot] jackjlli opened a new pull request #6181: Reduce pql queries from 10k to 500 in pinot-integration-tests module

2020-10-23 Thread GitBox
jackjlli opened a new pull request #6181: URL: https://github.com/apache/incubator-pinot/pull/6181 ## Description This PR: * removes all the queries that return empty result. * reduces pql queries from 10k to 500 in pinot-integration-tests module.

[incubator-pinot] branch refactor-integration-tests created (now d6d2c64)

2020-10-23 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch refactor-integration-tests in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at d6d2c64 Reduce pql queries from 10k to 500 This branch includes the following new

[GitHub] [incubator-pinot] fx19880617 commented on pull request #6127: Rewrite possible array aggregation functions to one level

2020-10-23 Thread GitBox
fx19880617 commented on pull request #6127: URL: https://github.com/apache/incubator-pinot/pull/6127#issuecomment-715083700 > High level question, should we overload `SUM`, `MIN`, `MAX` to directly work on MV columns? I feel it is much easier to use if these functions directly work on