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
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
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
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
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
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
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.
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 @@
+/*
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
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 @@
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
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
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
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
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
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
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
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
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
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
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 @@
+/**
+
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
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
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
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
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
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
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 @@
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 @@
*/
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
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
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
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
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,
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
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
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
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 -
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
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
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
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
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
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 @@
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
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
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
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
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
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 @@
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
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
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
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
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:
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 @@
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 @@
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 @@
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 @@
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 @@
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
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
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
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
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
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
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
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
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:
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
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
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
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
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
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
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
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-
77 matches
Mail list logo