[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #5082: Plug in the new routing introduced in #5046

2020-02-19 Thread GitBox
Jackie-Jiang opened a new pull request #5082: Plug in the new routing introduced in #5046 URL: https://github.com/apache/incubator-pinot/pull/5082 - Plug in the new routing for the new added routing features - Re-implement time boundary refresh as a general broker segment refresh to

[incubator-pinot] branch add_quickstart_into_pinot_admin updated (b894861 -> 943493e)

2020-02-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch add_quickstart_into_pinot_admin in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from b894861 Adding quickstart into Pinot admin command add 943493e Address

[GitHub] [incubator-pinot] akshayrai opened a new pull request #5081: [TE][subscription] Fix duplicate anomaly report issue and clean up subscription pipelines

2020-02-19 Thread GitBox
akshayrai opened a new pull request #5081: [TE][subscription] Fix duplicate anomaly report issue and clean up subscription pipelines URL: https://github.com/apache/incubator-pinot/pull/5081 Bug: Jira alerts were commenting the same anomalies repeatedly as the subscription watermarks were

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381552273 ## File path: pinot-core/src/test/java/org/apache/pinot/core/segment/index/loader/SegmentPreProcessorTest.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381543053 ## File path:

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381550680 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/store/ColumnIndexType.java ##

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381547721 ## File path:

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381543601 ## File path:

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381548461 ## File path:

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5080: Adding quickstart into Pinot admin command

2020-02-19 Thread GitBox
fx19880617 commented on a change in pull request #5080: Adding quickstart into Pinot admin command URL: https://github.com/apache/incubator-pinot/pull/5080#discussion_r381551777 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/QuickStartCommand.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381545230 ## File path:

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381546904 ## File path:

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5074: Support segment reload for text index

2020-02-19 Thread GitBox
mcvsubbu commented on a change in pull request #5074: Support segment reload for text index URL: https://github.com/apache/incubator-pinot/pull/5074#discussion_r381536639 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/creator/impl/V1Constants.java ##

[GitHub] [incubator-pinot] snleee commented on a change in pull request #5080: Adding quickstart into Pinot admin command

2020-02-19 Thread GitBox
snleee commented on a change in pull request #5080: Adding quickstart into Pinot admin command URL: https://github.com/apache/incubator-pinot/pull/5080#discussion_r381473361 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/QuickStartCommand.java

[GitHub] [incubator-pinot] haibow commented on issue #4954: Support schema evolution for consuming segments

2020-02-19 Thread GitBox
haibow commented on issue #4954: Support schema evolution for consuming segments URL: https://github.com/apache/incubator-pinot/pull/4954#issuecomment-588529071 Updates: - Treat newly added columns in the mutable segment as physical columns instead of virtual columns - Refactor

[GitHub] [incubator-pinot] codecov-io commented on issue #5083: Add test for supporting no dictionary on variable width columns in realtime segments

2020-02-19 Thread GitBox
codecov-io commented on issue #5083: Add test for supporting no dictionary on variable width columns in realtime segments URL: https://github.com/apache/incubator-pinot/pull/5083#issuecomment-588575067 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5083?src=pr=h1) Report

[incubator-pinot] branch adding_cluster_config_ops updated (4a739b1 -> 183d810)

2020-02-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch adding_cluster_config_ops in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 4a739b1 Adding Pinot Cluster Config API in controller and corresponding PinotAdmin

[GitHub] [incubator-pinot] jihaozh merged pull request #5060: [TE] add initial delay upon receiving events in scheduling

2020-02-19 Thread GitBox
jihaozh merged pull request #5060: [TE] add initial delay upon receiving events in scheduling URL: https://github.com/apache/incubator-pinot/pull/5060 This is an automated message from the Apache Git Service. To respond to

[incubator-pinot] branch master updated (4779341 -> 3c78a8c)

2020-02-19 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 4779341 Disbale text query realtime integration test (#5079) add 3c78a8c [TE] add initial delay upon

[GitHub] [incubator-pinot] jihaozh merged pull request #5059: [TE] Adjust dectection window using expected delay

2020-02-19 Thread GitBox
jihaozh merged pull request #5059: [TE] Adjust dectection window using expected delay URL: https://github.com/apache/incubator-pinot/pull/5059 This is an automated message from the Apache Git Service. To respond to the

[incubator-pinot] branch master updated (3c78a8c -> c613f46)

2020-02-19 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 3c78a8c [TE] add initial delay upon receiving events in scheduling (#5060) add c613f46 [TE] Adjust

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #5073: Adding PinotClusterConfig APIs in controller and corresponding Pinot Admin SubCommands

2020-02-19 Thread GitBox
codecov-io edited a comment on issue #5073: Adding PinotClusterConfig APIs in controller and corresponding Pinot Admin SubCommands URL: https://github.com/apache/incubator-pinot/pull/5073#issuecomment-586694558 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5073?src=pr=h1)

[GitHub] [incubator-pinot] siddharthteotia opened a new pull request #5083: Add test for supporting no dictionary on variable width columns in realtime segments

2020-02-19 Thread GitBox
siddharthteotia opened a new pull request #5083: Add test for supporting no dictionary on variable width columns in realtime segments URL: https://github.com/apache/incubator-pinot/pull/5083 This is an automated message

[incubator-pinot] branch hotfix-consuming-segments-no-dictionary created (now d715c4c)

2020-02-19 Thread mcvsubbu
This is an automated email from the ASF dual-hosted git repository. mcvsubbu pushed a change to branch hotfix-consuming-segments-no-dictionary in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at d715c4c Test and fix for supporting noDictionary String/Bytes in

[GitHub] [incubator-pinot] snleee commented on issue #5083: Add test for supporting no dictionary on variable width columns in realtime segments

2020-02-19 Thread GitBox
snleee commented on issue #5083: Add test for supporting no dictionary on variable width columns in realtime segments URL: https://github.com/apache/incubator-pinot/pull/5083#issuecomment-588559346 @mcvsubbu 's comment is based on the following code.

[GitHub] [incubator-pinot] fx19880617 merged pull request #5080: Adding quickstart into Pinot admin command

2020-02-19 Thread GitBox
fx19880617 merged pull request #5080: Adding quickstart into Pinot admin command URL: https://github.com/apache/incubator-pinot/pull/5080 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-pinot] branch master updated (c613f46 -> 7dfc5c1)

2020-02-19 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 c613f46 [TE] Adjust dectection window using expected delay (#5059) add 7dfc5c1 Adding quickstart

[incubator-pinot] 01/01: Test and fix for supporting noDictionary String/Bytes in consuming segments

2020-02-19 Thread mcvsubbu
This is an automated email from the ASF dual-hosted git repository. mcvsubbu pushed a commit to branch hotfix-consuming-segments-no-dictionary in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit d715c4c72490621ae6653837da4ff0c505516df5 Author: Subbu Subramaniam

[GitHub] [incubator-pinot] mcvsubbu commented on issue #5083: Add test for supporting no dictionary on variable width columns in realtime segments

2020-02-19 Thread GitBox
mcvsubbu commented on issue #5083: Add test for supporting no dictionary on variable width columns in realtime segments URL: https://github.com/apache/incubator-pinot/pull/5083#issuecomment-588563318 There is more to it than meets the eye. We dont want to just move this into the loop,