[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406357056 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[incubator-pinot] branch master updated: Address extra comments in #5224 (#5230)

2020-04-09 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie 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 535700b Address extra comments in

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #5230: Address extra comments in #5224

2020-04-09 Thread GitBox
Jackie-Jiang merged pull request #5230: Address extra comments in #5224 URL: https://github.com/apache/incubator-pinot/pull/5230 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-pinot] haibow opened a new pull request #5232: Fix logging in AvroUtil and add test

2020-04-09 Thread GitBox
haibow opened a new pull request #5232: Fix logging in AvroUtil and add test URL: https://github.com/apache/incubator-pinot/pull/5232 Now the logger in AvroUtil failed to get started. ``` 2020-04-09 11:31:06,618 main ERROR Unable to invoke factory method in class

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5232: Fix logging in AvroUtil and add test

2020-04-09 Thread GitBox
Jackie-Jiang commented on a change in pull request #5232: Fix logging in AvroUtil and add test URL: https://github.com/apache/incubator-pinot/pull/5232#discussion_r406407500 ## File path: pinot-plugins/pinot-input-format/pinot-avro-base/pom.xml ## @@ -50,5 +50,9 @@

[GitHub] [incubator-pinot] haibow commented on a change in pull request #5232: Fix logging in AvroUtil and add test

2020-04-09 Thread GitBox
haibow commented on a change in pull request #5232: Fix logging in AvroUtil and add test URL: https://github.com/apache/incubator-pinot/pull/5232#discussion_r406463693 ## File path:

[incubator-pinot] branch master updated: Handle queries like count(distinct ) (#5223)

2020-04-09 Thread xiangfu
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 16ac894 Handle queries like

[GitHub] [incubator-pinot] fx19880617 merged pull request #5223: Handle count distinct query in selection list.

2020-04-09 Thread GitBox
fx19880617 merged pull request #5223: Handle count distinct query in selection list. URL: https://github.com/apache/incubator-pinot/pull/5223 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
Jackie-Jiang commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406366285 ## File path:

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406372251 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5232: Fix logging in AvroUtil and add test

2020-04-09 Thread GitBox
Jackie-Jiang commented on a change in pull request #5232: Fix logging in AvroUtil and add test URL: https://github.com/apache/incubator-pinot/pull/5232#discussion_r406435149 ## File path:

[incubator-pinot] 01/01: [TE] add rate limiter on alert onboarding

2020-04-09 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a commit to branch rate-limiter-onboarding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 5db1df231c062e79bbf1a062bf72725338ff3060 Author: Jihao Zhang AuthorDate: Thu Apr 9 15:11:34

[incubator-pinot] branch rate-limiter-onboarding updated (9f0a0b1 -> 5db1df2)

2020-04-09 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch rate-limiter-onboarding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 9f0a0b1 [TE] add rate limiter on alert onboarding new 5db1df2 [TE] add rate limiter

[GitHub] [incubator-pinot] snleee commented on a change in pull request #5235: Fix the error message in RoutingManager

2020-04-09 Thread GitBox
snleee commented on a change in pull request #5235: Fix the error message in RoutingManager URL: https://github.com/apache/incubator-pinot/pull/5235#discussion_r406582247 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/routing/RoutingManager.java ## @@

[GitHub] [incubator-pinot] codecov-io commented on issue #5232: Fix logging in AvroUtil and add test

2020-04-09 Thread GitBox
codecov-io commented on issue #5232: Fix logging in AvroUtil and add test URL: https://github.com/apache/incubator-pinot/pull/5232#issuecomment-611788420 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5232?src=pr=h1) Report > Merging

[incubator-pinot] branch master updated (16ac894 -> 33fc485)

2020-04-09 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 16ac894 Handle queries like count(distinct ) (#5223) add 33fc485 fixing spark ingestion issue

[GitHub] [incubator-pinot] fx19880617 merged pull request #5216: fixing spark ingestion issue

2020-04-09 Thread GitBox
fx19880617 merged pull request #5216: fixing spark ingestion issue URL: https://github.com/apache/incubator-pinot/pull/5216 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #5232: Fix logging in AvroUtil and add test

2020-04-09 Thread GitBox
Jackie-Jiang merged pull request #5232: Fix logging in AvroUtil and add test URL: https://github.com/apache/incubator-pinot/pull/5232 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-pinot] branch master updated (33fc485 -> 08844de)

2020-04-09 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 33fc485 fixing spark ingestion issue (#5216) add 08844de Fix logging in AvroUtil and add test

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
chenboat commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406548795 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[GitHub] [incubator-pinot] fx19880617 commented on issue #4617: Adding CreateSegmentCommandV2 command

2020-04-09 Thread GitBox
fx19880617 commented on issue #4617: Adding CreateSegmentCommandV2 command URL: https://github.com/apache/incubator-pinot/pull/4617#issuecomment-611837460 closed this as we moved to new ingestion spec things This is an

[GitHub] [incubator-pinot] fx19880617 closed pull request #4617: Adding CreateSegmentCommandV2 command

2020-04-09 Thread GitBox
fx19880617 closed pull request #4617: Adding CreateSegmentCommandV2 command URL: https://github.com/apache/incubator-pinot/pull/4617 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
chenboat commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406509040 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
chenboat commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406509086 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
chenboat commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406550254 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[incubator-pinot] 01/01: [TE] add rate limiter on alert onboarding

2020-04-09 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a commit to branch rate-limiter-onboarding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 9bd02806284a686a1df3c355851dba920fcb4c4b Author: Jihao Zhang AuthorDate: Thu Apr 9 15:11:34

[incubator-pinot] branch rate-limiter-onboarding updated (b74bfce -> 9bd0280)

2020-04-09 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch rate-limiter-onboarding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard b74bfce [TE] add rate limiter on alert onboarding new 9bd0280 [TE] add rate limiter

[GitHub] [incubator-pinot] akshayrai opened a new pull request #5233: [TE][subscription] Subscription pipeline should use anomaly create ti…

2020-04-09 Thread GitBox
akshayrai opened a new pull request #5233: [TE][subscription] Subscription pipeline should use anomaly create ti… URL: https://github.com/apache/incubator-pinot/pull/5233 …me as watermark rather than the end time This PR merges the changes from #5152 addressing a bug, adding safety

[GitHub] [incubator-pinot] jihaozh opened a new pull request #5234: [TE] add rate limiter to batch alert onboarding

2020-04-09 Thread GitBox
jihaozh opened a new pull request #5234: [TE] add rate limiter to batch alert onboarding URL: https://github.com/apache/incubator-pinot/pull/5234 Add a rate limiter to the create alert API. The QPS permit can be set in Thirdeye configs. If the QPS is above the permit, it will return an

[incubator-pinot] branch upsert-pr-land updated (553598e -> 5306aa5)

2020-04-09 Thread jamesshao
This is an automated email from the ASF dual-hosted git repository. jamesshao pushed a change to branch upsert-pr-land in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 553598e enable multiple primary key support add 5306aa5 move upsert config mostly to

[incubator-pinot] branch rate-limiter-onboarding created (now b74bfce)

2020-04-09 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch rate-limiter-onboarding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at b74bfce [TE] add rate limiter on alert onboarding This branch includes the following new

[incubator-pinot] 01/01: [TE] add rate limiter on alert onboarding

2020-04-09 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a commit to branch rate-limiter-onboarding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit b74bfce88b228142d622b0c6813fa4d8dcc063b4 Author: Jihao Zhang AuthorDate: Thu Apr 9 15:11:34

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
chenboat commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406530371 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[incubator-pinot] 01/01: [TE] add rate limiter on alert onboarding

2020-04-09 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a commit to branch rate-limiter-onboarding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 9f0a0b16e84f76c6be5493cde1cf8229e9e978a2 Author: Jihao Zhang AuthorDate: Thu Apr 9 15:11:34

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406534983 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406535261 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[incubator-pinot] branch rate-limiter-onboarding updated (9bd0280 -> 9f0a0b1)

2020-04-09 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch rate-limiter-onboarding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 9bd0280 [TE] add rate limiter on alert onboarding new 9f0a0b1 [TE] add rate limiter

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments.

2020-04-09 Thread GitBox
mcvsubbu commented on a change in pull request #5221: Add a new server api for download of segments. URL: https://github.com/apache/incubator-pinot/pull/5221#discussion_r406536533 ## File path: pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java

[GitHub] [incubator-pinot] harleyjj opened a new pull request #5236: [TE] frontend - harleyjj/alert-details - add basic forecast display c…

2020-04-09 Thread GitBox
harleyjj opened a new pull request #5236: [TE] frontend - harleyjj/alert-details - add basic forecast display c… URL: https://github.com/apache/incubator-pinot/pull/5236 …apability to Alert Overview This is an automated

[incubator-pinot] branch master updated: Fix the error message in RoutingManager (#5235)

2020-04-09 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie 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 235b474 Fix the error message in

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5235: Fix the error message in RoutingManager

2020-04-09 Thread GitBox
Jackie-Jiang commented on a change in pull request #5235: Fix the error message in RoutingManager URL: https://github.com/apache/incubator-pinot/pull/5235#discussion_r406600986 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/routing/RoutingManager.java ##

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #5235: Fix the error message in RoutingManager

2020-04-09 Thread GitBox
Jackie-Jiang merged pull request #5235: Fix the error message in RoutingManager URL: https://github.com/apache/incubator-pinot/pull/5235 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] harleyjj opened a new pull request #5231: [TE] frontend - harleyjj/alert-details - show blob in place of graph …

2020-04-09 Thread GitBox
harleyjj opened a new pull request #5231: [TE] frontend - harleyjj/alert-details - show blob in place of graph … URL: https://github.com/apache/incubator-pinot/pull/5231 …when loading and to display error when no time series returned

[incubator-pinot] branch master updated (0a890fe -> 0e0eef4)

2020-04-09 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 0a890fe [TE] Remove MIGRATED_ALGORITHM type (#5172) add 0e0eef4 Update JVM settings for scripts

[GitHub] [incubator-pinot] fx19880617 merged pull request #5127: Update JVM settings for scripts

2020-04-09 Thread GitBox
fx19880617 merged pull request #5127: Update JVM settings for scripts URL: https://github.com/apache/incubator-pinot/pull/5127 This is an automated message from the Apache Git Service. To respond to the message, please log