kmozaid opened a new issue, #13982:
URL: https://github.com/apache/pinot/issues/13982
`select AirlineId, count(*) filter (where OriginStateName = 'xyz') from
airlineStats group by AirlineId limit 10` - This query does not produce any
result.
`select AirlineId, count(*) filter (where
Jackie-Jiang commented on code in PR #13966:
URL: https://github.com/apache/pinot/pull/13966#discussion_r1756179430
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/plan/server/ServerPlanRequestVisitor.java:
##
@@ -124,28 +128,41 @@ public Void visitFilter(Fil
yashmayya commented on code in PR #13981:
URL: https://github.com/apache/pinot/pull/13981#discussion_r1756163553
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/operator/HashJoinOperator.java:
##
@@ -389,18 +369,29 @@ private List
buildJoinedDataBlockDefault
kinok closed pull request #13440: [wip] fix: prevent 500 when taskTypeConfig is
null
URL: https://github.com/apache/pinot/pull/13440
--
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 to go to the specific comm
This is an automated email from the ASF dual-hosted git repository.
ankitsultana pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 26709271f2 Move untar method to BaseTaskExe
ankitsultana merged PR #13964:
URL: https://github.com/apache/pinot/pull/13964
--
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 to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
Jackie-Jiang commented on code in PR #13976:
URL: https://github.com/apache/pinot/pull/13976#discussion_r1756107098
##
pinot-common/src/main/java/org/apache/pinot/common/utils/helix/IdealStateGroupCommit.java:
##
@@ -0,0 +1,294 @@
+/**
+ * Licensed to the Apache Software Foundat
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/npm_and_yarn/pinot-controller/src/main/resources/multi-be58494012
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 39f1c44287 Bump path-to-regexp and react-rou
dependabot[bot] commented on PR #13974:
URL: https://github.com/apache/pinot/pull/13974#issuecomment-2345252816
OK, I won't notify you again about this release, but will get in touch when
a new version is available. You can also ignore all major, minor, or patch
releases for a dependency by
Jackie-Jiang closed pull request #13974: Bump path-to-regexp and
react-router-dom in /pinot-controller/src/main/resources
URL: https://github.com/apache/pinot/pull/13974
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
Jackie-Jiang merged PR #13980:
URL: https://github.com/apache/pinot/pull/13980
--
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 to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/npm_and_yarn/pinot-controller/src/main/resources/path-to-regexp-1.9.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 3ec8fe86ab Bump path-to-regexp in /pinot-con
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 58f2e97f7e Bump path-to-regexp in /pinot-controll
jasperjiaguo merged PR #13952:
URL: https://github.com/apache/pinot/pull/13952
--
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 to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
This is an automated email from the ASF dual-hosted git repository.
jiaguo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 604b244b64 Optimize Adaptive Server Selection (#1
praveenc7 commented on code in PR #13952:
URL: https://github.com/apache/pinot/pull/13952#discussion_r1755849086
##
pinot-broker/src/main/java/org/apache/pinot/broker/routing/instanceselector/ReplicaGroupInstanceSelector.java:
##
@@ -135,37 +137,31 @@ private Pair, Map>
selectS
mcvsubbu commented on code in PR #13584:
URL: https://github.com/apache/pinot/pull/13584#discussion_r1755282957
##
pinot-controller/src/main/java/org/apache/pinot/controller/validation/StorageQuotaChecker.java:
##
@@ -43,14 +44,19 @@ public class StorageQuotaChecker {
private
jasperjiaguo commented on code in PR #13952:
URL: https://github.com/apache/pinot/pull/13952#discussion_r1755824428
##
pinot-broker/src/main/java/org/apache/pinot/broker/routing/instanceselector/ReplicaGroupInstanceSelector.java:
##
@@ -135,37 +137,31 @@ private Pair, Map>
sele
praveenc7 commented on code in PR #13952:
URL: https://github.com/apache/pinot/pull/13952#discussion_r1755761361
##
pinot-broker/src/main/java/org/apache/pinot/broker/routing/instanceselector/ReplicaGroupInstanceSelector.java:
##
@@ -135,37 +137,30 @@ private Pair, Map>
selectS
codecov-commenter commented on PR #13981:
URL: https://github.com/apache/pinot/pull/13981#issuecomment-2344798110
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13981?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
Jackie-Jiang opened a new pull request, #13981:
URL: https://github.com/apache/pinot/pull/13981
In #13922 we added support to apply max rows limit to joined rows.
The intention is to protect operator from OOM on large `CROSS JOIN`, so we
want to limit the rows in memory (similar to the pr
rajagopr commented on code in PR #13646:
URL: https://github.com/apache/pinot/pull/13646#discussion_r1755592659
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/upload/ZKOperator.java:
##
@@ -276,6 +336,144 @@ private void processExistingSegment(String
tableNam
rajagopr commented on code in PR #13646:
URL: https://github.com/apache/pinot/pull/13646#discussion_r1755532575
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java:
##
@@ -2358,6 +2358,84 @@ public void assignTableSegment(Strin
rajagopr commented on code in PR #13646:
URL: https://github.com/apache/pinot/pull/13646#discussion_r1755531409
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/upload/ZKOperator.java:
##
@@ -110,6 +115,61 @@ public void completeSegmentOperations(String
tableNa
Jackie-Jiang commented on code in PR #13977:
URL: https://github.com/apache/pinot/pull/13977#discussion_r1755491769
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java:
##
@@ -298,6 +298,15 @@ protected BrokerResponse han
Jackie-Jiang commented on code in PR #13854:
URL: https://github.com/apache/pinot/pull/13854#discussion_r1755485136
##
.github/workflows/scripts/pr-tests/.pinot_tests_unit.sh:
##
@@ -48,5 +48,7 @@ if [ "$RUN_TEST_SET" == "2" ]; then
-pl '!pinot-query-planner' \
-pl '!p
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/npm_and_yarn/pinot-controller/src/main/resources/path-to-regexp-1.9.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 3ec8fe86ab Bump path-to-regexp in /pinot
dependabot[bot] opened a new pull request, #13980:
URL: https://github.com/apache/pinot/pull/13980
Bumps [path-to-regexp](https://github.com/pillarjs/path-to-regexp) from
1.8.0 to 1.9.0.
Release notes
Sourced from https://github.com/pillarjs/path-to-regexp/releases";>path-to-regexp
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.27.23
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 6be9e8a94f Bump software.amazon.awssdk:bom from 2.27.22 to 2.27.23
The r
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 83a71d2e39 Bump software.amazon.awssdk:bom from 2
Jackie-Jiang merged PR #13979:
URL: https://github.com/apache/pinot/pull/13979
--
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 to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/org.xerial.snappy-snappy-java-1.1.10.7
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 96be9918ec Bump org.xerial.snappy:snappy-java from 1.1.10.6 to 1.1.10
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new ca7abbf176 Bump org.xerial.snappy:snappy-java fro
Jackie-Jiang merged PR #13978:
URL: https://github.com/apache/pinot/pull/13978
--
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 to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
jasperjiaguo commented on code in PR #13952:
URL: https://github.com/apache/pinot/pull/13952#discussion_r1755232514
##
pinot-broker/src/main/java/org/apache/pinot/broker/routing/instanceselector/ReplicaGroupInstanceSelector.java:
##
@@ -135,37 +137,30 @@ private Pair, Map>
sele
jasperjiaguo commented on code in PR #13952:
URL: https://github.com/apache/pinot/pull/13952#discussion_r1755231339
##
pinot-broker/src/main/java/org/apache/pinot/broker/routing/instanceselector/ReplicaGroupInstanceSelector.java:
##
@@ -135,37 +137,30 @@ private Pair, Map>
sele
rfscholte commented on code in PR #13854:
URL: https://github.com/apache/pinot/pull/13854#discussion_r1755074144
##
pom.xml:
##
@@ -354,6 +354,55 @@
-Xms4g -Xmx4g -Dlog4j2.configurationFile=log4j2.xml
+
+ codecoverage
+
+
+
rfscholte commented on code in PR #13854:
URL: https://github.com/apache/pinot/pull/13854#discussion_r1754970138
##
reports/pom.xml:
##
@@ -0,0 +1,355 @@
+
+
+http://maven.apache.org/POM/4.0.0";
+ xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
+ xsi:schem
rfscholte commented on code in PR #13854:
URL: https://github.com/apache/pinot/pull/13854#discussion_r1754947033
##
pinot-integration-tests/pom.xml:
##
@@ -36,6 +36,25 @@
+
Review Comment:
I've adjusted it a bit, by adding the goal+phase, so it can become part
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1754564993
##
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/TransformOperator.java:
##
@@ -86,7 +88,17 @@ public String toExplainString() {
}
protected
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1754550758
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/service/dispatch/DispatchClient.java:
##
@@ -50,12 +51,17 @@ public ManagedChannel getChannel() {
}
p
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.27.23
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 6be9e8a94f Bump software.amazon.awssdk:bom from 2.27.22 to 2.27.23
N
dependabot[bot] opened a new pull request, #13979:
URL: https://github.com/apache/pinot/pull/13979
Bumps software.amazon.awssdk:bom from 2.27.22 to 2.27.23.
[
from 1.1.10.6 to 1.1.10.7.
Release notes
Sourced from https://github.com/xerial/snappy-java/releases";>org
codecov-commenter commented on PR #13977:
URL: https://github.com/apache/pinot/pull/13977#issuecomment-2343290995
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13977?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
yashmayya opened a new pull request, #13977:
URL: https://github.com/apache/pinot/pull/13977
- Currently, query null handling support can be enabled via the query option
`enableNullHandling` (see
https://docs.pinot.apache.org/developers/advanced/null-value-support#advanced-null-handling-sup
xiangfu0 commented on code in PR #13976:
URL: https://github.com/apache/pinot/pull/13976#discussion_r1753856067
##
pinot-common/src/main/java/org/apache/pinot/common/utils/helix/IdealStateGroupCommit.java:
##
@@ -0,0 +1,298 @@
+/**
+ * Licensed to the Apache Software Foundation
rfscholte commented on code in PR #13930:
URL: https://github.com/apache/pinot/pull/13930#discussion_r1753819807
##
pinot-spi/pom.xml:
##
@@ -36,6 +36,82 @@
+
+org.apache.maven.plugins
+maven-dependency-plugin
+3.7.1
+
+
codecov-commenter commented on PR #13976:
URL: https://github.com/apache/pinot/pull/13976#issuecomment-2343053302
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13976?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
rfscholte commented on code in PR #13930:
URL: https://github.com/apache/pinot/pull/13930#discussion_r1753669384
##
pinot-spi/pom.xml:
##
@@ -36,6 +36,82 @@
+
+org.apache.maven.plugins
+maven-dependency-plugin
+3.7.1
+
+
rfscholte commented on PR #13969:
URL: https://github.com/apache/pinot/pull/13969#issuecomment-2343032239
I've also created https://issues.apache.org/jira/browse/CALCITE-6577 . Maybe
they come with a solution which makes it easier to extend the template (without
manually having to patch the
gortiz commented on code in PR #13930:
URL: https://github.com/apache/pinot/pull/13930#discussion_r1753575891
##
pinot-spi/pom.xml:
##
@@ -36,6 +36,82 @@
+
+org.apache.maven.plugins
+maven-dependency-plugin
+3.7.1
+
+
+
gortiz commented on PR #13969:
URL: https://github.com/apache/pinot/pull/13969#issuecomment-2342971443
> What does the fmpp-maven-plugin do? Quick googling and I can't find any
decent documentation on it.
This is a plugin we use to generate code (specifically, our custom Calcite
pars
KKcorps commented on code in PR #13976:
URL: https://github.com/apache/pinot/pull/13976#discussion_r1753527635
##
pinot-common/src/main/java/org/apache/pinot/common/utils/helix/IdealStateGroupCommit.java:
##
@@ -0,0 +1,298 @@
+/**
+ * Licensed to the Apache Software Foundation (
KKcorps commented on code in PR #13976:
URL: https://github.com/apache/pinot/pull/13976#discussion_r1753524472
##
pinot-common/src/main/java/org/apache/pinot/common/utils/helix/IdealStateGroupCommit.java:
##
@@ -0,0 +1,298 @@
+/**
+ * Licensed to the Apache Software Foundation (
xiangfu0 opened a new pull request, #13976:
URL: https://github.com/apache/pinot/pull/13976
We observed IdealState update becomes bottleneck for large table with high
throughput segments update.
This PR shamelessly borrowed the idea from:
https://github.com/apache/helix/blob/master/helix
gortiz commented on code in PR #13966:
URL: https://github.com/apache/pinot/pull/13966#discussion_r1753454415
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/plan/server/ServerPlanRequestVisitor.java:
##
@@ -124,28 +128,41 @@ public Void visitFilter(FilterNod
gortiz commented on code in PR #13955:
URL: https://github.com/apache/pinot/pull/13955#discussion_r1753321639
##
pinot-core/src/main/java/org/apache/pinot/core/accounting/PerQueryCPUMemAccountantFactory.java:
##
@@ -157,6 +160,11 @@ public
PerQueryCPUMemResourceUsageAccountant(
59 matches
Mail list logo
