[incubator-pinot] annotated tag release-0.4.0-rc4 updated (1ab389e -> 3747154)

2020-05-29 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to annotated tag release-0.4.0-rc4 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. *** WARNING: tag release-0.4.0-rc4 was modified! *** from 1ab389e (commit) to 3747154

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

2020-05-29 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch release-0.4.0-rc4 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/release-0.4.0-rc4 by this push: new b8b5d8b [maven-

[incubator-pinot] branch release-0.4.0-rc4 created (now 1ab389e)

2020-05-29 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch release-0.4.0-rc4 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 1ab389e [maven-release-plugin] prepare release release-0.4.0-rc4 This branch includes the foll

[incubator-pinot] 01/01: [maven-release-plugin] prepare release release-0.4.0-rc4

2020-05-29 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch release-0.4.0-rc4 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 1ab389e5c09f8abc20cd4d4eed10df2f6cac4638 Author: Xiang Fu AuthorDate: Fri May 29 22:23:32 2020 -070

[GitHub] [incubator-pinot] vincentchenjl commented on a change in pull request #5435: [TE] clean up decprecated/unused code

2020-05-29 Thread GitBox
vincentchenjl commented on a change in pull request #5435: URL: https://github.com/apache/incubator-pinot/pull/5435#discussion_r432790440 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/AnomalyResource.java ## @@ -101,21 +99,18

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang merged pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[incubator-pinot] branch master updated: Refactor DistinctTable to use PriorityQueue based algorithm (#5451)

2020-05-29 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 44a1e2e Refactor DistinctTable to use

[GitHub] [incubator-pinot] vincentchenjl commented on a change in pull request #5435: [TE] clean up decprecated/unused code

2020-05-29 Thread GitBox
vincentchenjl commented on a change in pull request #5435: URL: https://github.com/apache/incubator-pinot/pull/5435#discussion_r432789651 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/ThirdEyeDashboardApplication.java ## @@ -160,14 +15

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5440: Add GenericTransformFunction wrapper for simple ScalarFunctions

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5440: URL: https://github.com/apache/incubator-pinot/pull/5440#discussion_r432782384 ## File path: pinot-common/pom.xml ## @@ -33,6 +33,7 @@ https://pinot.apache.org/ ${basedir}/.. +0.9.11 Review comment:

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

2020-05-29 Thread haibow
This is an automated email from the ASF dual-hosted git repository. haibow pushed a commit to branch release-0.4.0-rc3 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/release-0.4.0-rc3 by this push: new 46863b1 [maven-r

[incubator-pinot] annotated tag release-0.4.0-rc3 updated (c3f36c2 -> bf16d5d)

2020-05-29 Thread haibow
This is an automated email from the ASF dual-hosted git repository. haibow pushed a change to annotated tag release-0.4.0-rc3 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. *** WARNING: tag release-0.4.0-rc3 was modified! *** from c3f36c2 (commit) to bf16d5d

[incubator-pinot] 02/02: [maven-release-plugin] prepare release release-0.4.0-rc3

2020-05-29 Thread haibow
This is an automated email from the ASF dual-hosted git repository. haibow pushed a commit to branch release-0.4.0-rc3 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit c3f36c28b37e4ea216433be3d6eb8b5dc3603f3f Author: Haibo Wang AuthorDate: Fri May 29 17:13:32 2020 -07

[incubator-pinot] branch release-0.4.0-rc3 created (now c3f36c2)

2020-05-29 Thread haibow
This is an automated email from the ASF dual-hosted git repository. haibow pushed a change to branch release-0.4.0-rc3 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at c3f36c2 [maven-release-plugin] prepare release release-0.4.0-rc3 This branch includes the follo

[incubator-pinot] 01/02: Add license

2020-05-29 Thread haibow
This is an automated email from the ASF dual-hosted git repository. haibow pushed a commit to branch release-0.4.0-rc3 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 7f65bfe5e87dc3a3b2e743c991c534ab34a7aeac Author: Haibo Wang AuthorDate: Fri May 29 01:10:14 2020 -07

[GitHub] [incubator-pinot] Jackie-Jiang commented on pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang commented on pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#issuecomment-636230181 @siddharthteotia Addressed all the comments This is an automated message from the Apache Git Servic

[incubator-pinot] branch master updated: Adding files generated by running quickstart to gitignore (#5441)

2020-05-29 Thread kishoreg
This is an automated email from the ASF dual-hosted git repository. kishoreg 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 fb5b75f Adding files generated by ru

[GitHub] [incubator-pinot] kishoreg merged pull request #5441: Adding files generated by running quickstart to gitignore

2020-05-29 Thread GitBox
kishoreg merged pull request #5441: URL: https://github.com/apache/incubator-pinot/pull/5441 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] Jackie-Jiang commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432775396 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/customobject/DistinctTable.java ## @@ -0,0 +1,297 @@

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432774922 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/customobject/DistinctTable.java ## @@ -0,0 +1,297 @@

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432774854 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/customobject/DistinctTable.java ## @@ -0,0 +1,297 @@

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432774276 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/customobject/DistinctTable.java ## @@ -0,0 +1,297 @@

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432774378 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/customobject/DistinctTable.java ## @@ -0,0 +1,297 @@

[incubator-pinot] branch hotfix_chunkwriter_realtime updated (48553ab -> f6a3476)

2020-05-29 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch hotfix_chunkwriter_realtime in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 48553ab remove master branch restriction in travis.yml new f6a3476 remove master b

[incubator-pinot] 01/01: remove master branch restriction in travis.yml

2020-05-29 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch hotfix_chunkwriter_realtime in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit f6a34765160212e31e9e14f3948511747c9ef541 Author: Siddharth Teotia AuthorDate: Thu May 28 23:

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5461: Adding Support for SQL CASE Statement

2020-05-29 Thread GitBox
fx19880617 commented on a change in pull request #5461: URL: https://github.com/apache/incubator-pinot/pull/5461#discussion_r432769120 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -610,6 +611,25 @@ private static Expression

[incubator-pinot] branch support_case_when_statement updated (43bb868 -> 8e00513)

2020-05-29 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch support_case_when_statement in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 43bb868 Checks on then statements result type add 8e00513 Not allowing aggregat

[incubator-pinot] branch support_case_when_statement updated (385a8b8 -> 43bb868)

2020-05-29 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch support_case_when_statement in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 385a8b8 Adding transform function support for case-when-else add 43bb868 Checks

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432742947 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/customobject/DistinctTable.java ## @@ -0,0 +1,297 @@

[GitHub] [incubator-pinot] siddharthteotia edited a comment on pull request #5440: Add GenericTransformFunction wrapper for simple ScalarFunctions

2020-05-29 Thread GitBox
siddharthteotia edited a comment on pull request #5440: URL: https://github.com/apache/incubator-pinot/pull/5440#issuecomment-636196791 > @fx19880617 @siddharthteotia Should I add tests in CalciteSQL for all the functions? @KKcorps , sorry missed this. Yes, the query compilation test

[GitHub] [incubator-pinot] siddharthteotia edited a comment on pull request #5440: Add GenericTransformFunction wrapper for simple ScalarFunctions

2020-05-29 Thread GitBox
siddharthteotia edited a comment on pull request #5440: URL: https://github.com/apache/incubator-pinot/pull/5440#issuecomment-636196791 > @fx19880617 @siddharthteotia Should I add tests in CalciteSQL for all the functions? @KKcorps , sorry missed this. Yes, the query compilation test

[GitHub] [incubator-pinot] siddharthteotia commented on pull request #5440: Add GenericTransformFunction wrapper for simple ScalarFunctions

2020-05-29 Thread GitBox
siddharthteotia commented on pull request #5440: URL: https://github.com/apache/incubator-pinot/pull/5440#issuecomment-636196791 > @fx19880617 @siddharthteotia Should I add tests in CalciteSQL for all the functions? @KKcorps , sorry missed seeing this. Yes, the query compilation test

[GitHub] [incubator-pinot] KKcorps edited a comment on pull request #5440: Add GenericTransformFunction wrapper for simple ScalarFunctions

2020-05-29 Thread GitBox
KKcorps edited a comment on pull request #5440: URL: https://github.com/apache/incubator-pinot/pull/5440#issuecomment-634088300 @fx19880617 @siddharthteotia Should I add tests in CalciteSQL for all the functions? Th

[GitHub] [incubator-pinot] KKcorps edited a comment on pull request #5440: Add GenericTransformFunction wrapper for simple ScalarFunctions

2020-05-29 Thread GitBox
KKcorps edited a comment on pull request #5440: URL: https://github.com/apache/incubator-pinot/pull/5440#issuecomment-634088300 @fx19880617 @siddharthteotia Should I add tests in CalciteSQL for all the functions? @KKcorps , sorry missed seeing this. Yes, the query compilation tests

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432323335 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/dociditerators/SVScanDocIdIterator.java ## @@ -18,238 +18,138 @@

[incubator-pinot] branch master updated: Faster bit unpacking (Part 1) (#5409)

2020-05-29 Thread siddteotia
This is an automated email from the ASF dual-hosted git repository. siddteotia 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 b40dd99 Faster bit unpacking (Part

[GitHub] [incubator-pinot] siddharthteotia merged pull request #5409: Faster bit unpacking (Part 1)

2020-05-29 Thread GitBox
siddharthteotia merged pull request #5409: URL: https://github.com/apache/incubator-pinot/pull/5409 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] siddharthteotia commented on pull request #5409: Faster bit unpacking (Part 1)

2020-05-29 Thread GitBox
siddharthteotia commented on pull request #5409: URL: https://github.com/apache/incubator-pinot/pull/5409#issuecomment-636192732 Merging it. Follow-ups coming next: - Wire with the reader and writer. Currently the fast methods here can be used for existing reader and writer i

[GitHub] [incubator-pinot] xiaohui-sun merged pull request #5447: [TE] frontend - harleyjj/rca - update frontend for new AC event format

2020-05-29 Thread GitBox
xiaohui-sun merged pull request #5447: URL: https://github.com/apache/incubator-pinot/pull/5447 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[incubator-pinot] branch master updated: [TE] frontend - harleyjj/rca - update frontend for new AC event format (#5447)

2020-05-29 Thread xhsun
This is an automated email from the ASF dual-hosted git repository. xhsun 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 79260b3 [TE] frontend - harleyjj/rca -

[incubator-pinot] branch master updated (6bfcacb -> 71ce427)

2020-05-29 Thread xhsun
This is an automated email from the ASF dual-hosted git repository. xhsun pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 6bfcacb [Cleanup] Merge RealtimeSegmentOnlineOfflineStateModel and SegmentOnlineOfflineStateModel in Common

[incubator-pinot] branch master updated: [TE] frontend - harleyjj/components - remove dead components (#5466)

2020-05-29 Thread xhsun
This is an automated email from the ASF dual-hosted git repository. xhsun 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 2807584 [TE] frontend - harleyjj/compon

[GitHub] [incubator-pinot] xiaohui-sun merged pull request #5466: [TE] frontend - harleyjj/components - remove dead components

2020-05-29 Thread GitBox
xiaohui-sun merged pull request #5466: URL: https://github.com/apache/incubator-pinot/pull/5466 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [incubator-pinot] xiaohui-sun merged pull request #5460: [TE] frontend - harleyjj/packages - remove bower from frontend

2020-05-29 Thread GitBox
xiaohui-sun merged pull request #5460: URL: https://github.com/apache/incubator-pinot/pull/5460 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432711634 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctAggregationFunction.java ## @@ -123,21 +120,

[GitHub] [incubator-pinot] Jackie-Jiang commented on pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
Jackie-Jiang commented on pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#issuecomment-636166279 This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432704090 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/docidsets/AndDocIdSet.java ## @@ -0,0 +1,156 @@ +/** + * Licensed to t

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432700876 ## File path: pinot-core/src/main/java/org/apache/pinot/core/common/BlockDocIdIterator.java ## @@ -25,25 +25,16 @@ public interface BlockDo

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
kishoreg commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432699285 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/dociditerators/SVScanDocIdIterator.java ## @@ -18,238 +18,138 @@ */ pac

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432695654 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/dociditerators/SVScanDocIdIterator.java ## @@ -18,238 +18,138 @@ */

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432692609 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/dociditerators/ScanBasedDocIdIterator.java ## @@ -24,22 +24,18 @@

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #5465: Support distinctCountRawThetaSketch aggregation that returns serialized sketch.

2020-05-29 Thread GitBox
kishoreg commented on a change in pull request #5465: URL: https://github.com/apache/incubator-pinot/pull/5465#discussion_r432689807 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/AggregationFunctionFactory.java ## @@ -137,6 +137,8 @@ p

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #5461: Adding Support for SQL CASE Statement

2020-05-29 Thread GitBox
kishoreg commented on a change in pull request #5461: URL: https://github.com/apache/incubator-pinot/pull/5461#discussion_r432688497 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -610,6 +611,25 @@ private static Expression to

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432681482 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/filter/BitmapBasedFilterOperator.java ## @@ -25,62 +25,73 @@ import o

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432678859 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctAggregationFunction.java ## @@ -123,21 +1

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432679650 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/customobject/DistinctTable.java ## @@ -0,0 +1,297

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432679222 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/customobject/DistinctTable.java ## @@ -0,0 +1,297

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5451: Refactor DistinctTable to use PriorityQueue based algorithm

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5451: URL: https://github.com/apache/incubator-pinot/pull/5451#discussion_r432678859 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctAggregationFunction.java ## @@ -123,21 +1

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432678997 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/filter/SortedIndexBasedFilterOperator.java ## @@ -0,0 +1,145 @@ +/** +

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #5459: [Cleanup] Merge RealtimeSegmentOnlineOfflineStateModel and SegmentOnlineOfflineStateModel in CommonConstants

2020-05-29 Thread GitBox
Jackie-Jiang merged pull request #5459: URL: https://github.com/apache/incubator-pinot/pull/5459 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[incubator-pinot] branch master updated: [Cleanup] Merge RealtimeSegmentOnlineOfflineStateModel and SegmentOnlineOfflineStateModel in CommonConstants (#5459)

2020-05-29 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 6bfcacb [Cleanup] Merge RealtimeSegmen

[incubator-pinot] branch master updated: Remove master branch restriction (#5467)

2020-05-29 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli 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 7310ffb Remove master branch restriction

[GitHub] [incubator-pinot] jackjlli merged pull request #5467: Remove master branch restriction to trigger Travis builds

2020-05-29 Thread GitBox
jackjlli merged pull request #5467: URL: https://github.com/apache/incubator-pinot/pull/5467 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] jackjlli commented on pull request #5467: Remove master branch restriction to trigger Travis builds

2020-05-29 Thread GitBox
jackjlli commented on pull request #5467: URL: https://github.com/apache/incubator-pinot/pull/5467#issuecomment-636049216 > Does this change will start to run Travis job for all existing branches? (we have `155` branches as of writing this comment). We should be able to trigger in ad-hoc b

[GitHub] [incubator-pinot] snleee edited a comment on pull request #5467: Remove master branch restriction to trigger Travis builds

2020-05-29 Thread GitBox
snleee edited a comment on pull request #5467: URL: https://github.com/apache/incubator-pinot/pull/5467#issuecomment-635850318 Does this change will start to run Travis job for all existing branches? (we have `155` branches as of writing this comment). We should be able to trigger in ad-ho

[GitHub] [incubator-pinot] snleee commented on pull request #5467: Remove master branch restriction to trigger Travis builds

2020-05-29 Thread GitBox
snleee commented on pull request #5467: URL: https://github.com/apache/incubator-pinot/pull/5467#issuecomment-635850318 Does this will start to run Travis job for all existing branches? (we have `155` branches as of writing this comment). We should be able to trigger in ad-hoc basis for br

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432331313 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/docidsets/AndDocIdSet.java ## @@ -0,0 +1,156 @@ +/** + * Licensed t

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432323606 ## File path: pinot-core/src/main/java/org/apache/pinot/core/common/BlockDocIdSet.java ## @@ -18,9 +18,15 @@ */ package org.apache.pin

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432323266 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/dociditerators/ScanBasedDocIdIterator.java ## @@ -24,22 +24,18 @@

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432322094 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/filter/BitmapBasedFilterOperator.java ## @@ -25,62 +25,73 @@ impor

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
siddharthteotia commented on a change in pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r432321846 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/filter/SortedIndexBasedFilterOperator.java ## @@ -0,0 +1,145 @@ +/*

[incubator-pinot] tag release-0.4.0-rc2 created (now 0ee9083)

2020-05-29 Thread haibow
This is an automated email from the ASF dual-hosted git repository. haibow pushed a change to tag release-0.4.0-rc2 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 0ee9083 (commit) No new revisions were added by this update.

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5461: Adding Support for SQL CASE Statement

2020-05-29 Thread GitBox
fx19880617 commented on a change in pull request #5461: URL: https://github.com/apache/incubator-pinot/pull/5461#discussion_r432313174 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/BinaryOperatorTransformFunction.java ## @@ -0,0 +1,11

[incubator-pinot] branch support_case_when_statement updated (5667776 -> 385a8b8)

2020-05-29 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch support_case_when_statement in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 5667776 Adding transform function support for case-when-else add 385a8b8 Adding

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5461: Adding Support for SQL CASE Statement

2020-05-29 Thread GitBox
fx19880617 commented on a change in pull request #5461: URL: https://github.com/apache/incubator-pinot/pull/5461#discussion_r432305059 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/BinaryOperatorTransformFunction.java ## @@ -0,0 +1,11

[GitHub] [incubator-pinot] chenboat edited a comment on pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
chenboat edited a comment on pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#issuecomment-635802473 > @chenboat There is no new added major classes. Most of the changes are making the classes compatible with the filter interface change, so it is very hard to br

[GitHub] [incubator-pinot] chenboat edited a comment on pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
chenboat edited a comment on pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#issuecomment-635802473 > @chenboat There is no new added major classes. Most of the changes are making the classes compatible with the filter interface change, so it is very hard to br

[GitHub] [incubator-pinot] chenboat commented on pull request #5444: Enhance and simplify the filtering

2020-05-29 Thread GitBox
chenboat commented on pull request #5444: URL: https://github.com/apache/incubator-pinot/pull/5444#issuecomment-635802473 > @chenboat There is no new added major classes. Most of the changes are making the classes compatible with the filter interface change, so it is very hard to break the