RSashimi commented on issue #15459:
URL: https://github.com/apache/pinot/issues/15459#issuecomment-2803863514
hi, i can take this
--
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
yashmayya commented on code in PR #15525:
URL: https://github.com/apache/pinot/pull/15525#discussion_r2043619684
##
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/MinMaxRangeAggregationFunction.java:
##
@@ -81,10 +84,13 @@ public void aggregate(int len
Jackie-Jiang commented on code in PR #15419:
URL: https://github.com/apache/pinot/pull/15419#discussion_r2042894711
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##
@@ -518,15 +518,13 @@ public void addConsumingSegment(Str
This is an automated email from the ASF dual-hosted git repository.
gortiz pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from fb02f04516 Fix Direct Memory OOM on Server (#15335)
add 883d8ee36a Substitute TransferableBlock with MseBlock (#15
Jackie-Jiang merged PR #15436:
URL: https://github.com/apache/pinot/pull/15436
--
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
lnbest0707-uber opened a new pull request, #15542:
URL: https://github.com/apache/pinot/pull/15542
`feature` `ingestion` realtime`
Add a new `mergedTextIndexPrefixToExclude` config field to
SchemaConformingTransformerConfig.
Before this, we had `_mergedTextIndexPathToExclude` but t
bziobrowski commented on code in PR #15508:
URL: https://github.com/apache/pinot/pull/15508#discussion_r2035549838
##
pinot-core/src/test/java/org/apache/pinot/core/operator/transform/function/BaseTransformFunctionTest.java:
##
@@ -311,18 +311,7 @@ public void setUp()
.
Jackie-Jiang commented on code in PR #15525:
URL: https://github.com/apache/pinot/pull/15525#discussion_r2043248743
##
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/MinMaxRangeAggregationFunction.java:
##
@@ -81,10 +84,13 @@ public void aggregate(int
Jackie-Jiang commented on issue #15503:
URL: https://github.com/apache/pinot/issues/15503#issuecomment-2803430640
Are you running Pinot `1.3.0`? The behavior should be fixed in #14258 where
you should only need to set it in one place
--
This is an automated message from the Apache Git Ser
codecov-commenter commented on PR #15544:
URL: https://github.com/apache/pinot/pull/15544#issuecomment-2803355091
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15544?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
ankitsultana merged PR #15539:
URL: https://github.com/apache/pinot/pull/15539
--
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.
ankitsultana pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 04c8a4f6ee [multistage] Add Leaf Stage Worker Assignment / Boundary /
Agg Rules (#15481)
add 40a5b531a5 [ti
somandal opened a new pull request, #15544:
URL: https://github.com/apache/pinot/pull/15544
This is just a test PR to combine changes from
https://github.com/apache/pinot/pull/15063 and
https://github.com/apache/pinot/pull/15519 to test them together
--
This is an automated message from
pengding-stripe opened a new issue, #15543:
URL: https://github.com/apache/pinot/issues/15543
Pinot has this
[check](https://github.com/apache/pinot/blob/master/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/instance/InstanceTagPoolSelector.java#L110)
to en
itschrispeck commented on issue #15056:
URL: https://github.com/apache/pinot/issues/15056#issuecomment-2803249791
I was recently exploring this change too, for a slightly different reason -
we use `llcpartitionconsuming` metric to alert when ingestion is stopped for a
long period of time. I
somandal commented on code in PR #15518:
URL: https://github.com/apache/pinot/pull/15518#discussion_r2043088823
##
pinot-common/src/main/java/org/apache/pinot/common/metrics/ControllerGauge.java:
##
@@ -205,7 +205,12 @@ public enum ControllerGauge implements
AbstractMetrics.Gau
lnbest0707-uber commented on code in PR #15542:
URL: https://github.com/apache/pinot/pull/15542#discussion_r2043019125
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/SchemaConformingTransformer.java:
##
Review Comment:
tune the format
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 a762765f67 Bump software.amazon.awssdk:bom from 2
codecov-commenter commented on PR #15539:
URL: https://github.com/apache/pinot/pull/15539#issuecomment-2803059533
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15539?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
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 04c8a4f6ee [multistage] Add Leaf Stage Work
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/babel/runtime-7.27.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 7c33f0cace Bump @babel/runtime in /pinot-con
Jackie-Jiang commented on code in PR #15419:
URL: https://github.com/apache/pinot/pull/15419#discussion_r2042895926
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##
@@ -518,15 +518,13 @@ public void addConsumingSegment(Str
Jackie-Jiang merged PR #15521:
URL: https://github.com/apache/pinot/pull/15521
--
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.apache.commons-commons-text-1.13.1
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 30a404bc44 Bump org.apache.commons:commons-text from 1.13.0 to 1.13.1
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/pinot.git
from f1fda590ed Bump @babel/runtime in /pinot-controller/src/main/resources
(#15541)
add 4bbe9fdca6 Bump org.apache.co
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/pinot.git
from f875678c49 Bump commons-io:commons-io from 2.18.0 to 2.19.0 (#15536)
add f1fda590ed Bump @babel/runtime in /pinot-
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/commons-io-commons-io-2.19.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 4a39caf634 Revert test change
The revisions that were on this branch are still
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 e03764a1f6 Table Rebalance Logger (#15436)
e03764
Jackie-Jiang merged PR #15536:
URL: https://github.com/apache/pinot/pull/15536
--
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 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 f875678c49 Bump commons-io:commons-io from 2.18.0
Jackie-Jiang commented on PR #15515:
URL: https://github.com/apache/pinot/pull/15515#issuecomment-2802798359
> The current design document mentions MIN( offline table timestamp ...) as
time boundary strategy. Is your suggestion to have field in the logical table
config to specify the time b
gortiz merged PR #15245:
URL: https://github.com/apache/pinot/pull/15245
--
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.apach
albertobastos commented on code in PR #15533:
URL: https://github.com/apache/pinot/pull/15533#discussion_r2042796526
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/QueryErrorCode.java:
##
@@ -57,6 +57,7 @@ public enum QueryErrorCode {
UNKNOWN_COLUMN(710, "UnknownCo
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/commons-io-commons-io-2.19.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
from a52b347684 Bump commons-io:commons-io from 2.11.0 to 2.19.0
add 4a39caf634
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/babel/runtime-7.27.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 7c33f0cace Bump @babel/runtime in /pinot
dependabot[bot] opened a new pull request, #15541:
URL: https://github.com/apache/pinot/pull/15541
Bumps
[@babel/runtime](https://github.com/babel/babel/tree/HEAD/packages/babel-runtime)
from 7.26.0 to 7.27.0.
Release notes
Sourced from https://github.com/babel/babel/releases";>@
Jackie-Jiang merged PR #15535:
URL: https://github.com/apache/pinot/pull/15535
--
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/software.amazon.awssdk-bom-2.31.21
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 9f2e518ae5 Bump software.amazon.awssdk:bom from 2.31.19 to 2.31.21
The r
ssharman-cloud opened a new issue, #15540:
URL: https://github.com/apache/pinot/issues/15540
In some strange scenarios, Pinot Server is keep on crashing with
"CrashLoopBackOff" error, zookeeper is not able to communicate to the Pinot
Server. Find below few observation from our side, but
gortiz opened a new pull request, #15538:
URL: https://github.com/apache/pinot/pull/15538
Some weeks ago, I modified QueryEnvironment to include a new class called
CompiledQuery. This class contains the result of parsing, validating,
transforming into RelNode and then optimizing the query.
J-HowHuang commented on code in PR #15518:
URL: https://github.com/apache/pinot/pull/15518#discussion_r2042622051
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/ZkBasedTableRebalanceObserver.java:
##
@@ -122,6 +125,7 @@ public void onTrigger(T
rseetham opened a new pull request, #15539:
URL: https://github.com/apache/pinot/pull/15539
`cleanup`
--
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
destrex271 commented on issue #15460:
URL: https://github.com/apache/pinot/issues/15460#issuecomment-2802390184
Hi @abhioncbr , this might take me some time to wrap up. I hope that's not
an immediate blocker?
--
This is an automated message from the Apache Git Service.
To respond to the m
ankugit commented on issue #14984:
URL: https://github.com/apache/pinot/issues/14984#issuecomment-2801622660
I faced the very same issue while working on similar configs and scale. As
an workaround as @Jackie-Jiang mentioned we moved the sparse columns to JSON
columns along with enabling co
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/pinot.git
from 1515f49c4c Fix semi-join / pipeline breaker on BYTES and BIG_DECIMAL
(#15531)
add 0bfc4542a0 Add extension point
gortiz commented on code in PR #15533:
URL: https://github.com/apache/pinot/pull/15533#discussion_r2042371603
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/QueryErrorCode.java:
##
@@ -57,6 +57,7 @@ public enum QueryErrorCode {
UNKNOWN_COLUMN(710, "UnknownColumnErr
xiangfu0 merged PR #15508:
URL: https://github.com/apache/pinot/pull/15508
--
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.apa
albertobastos commented on code in PR #15533:
URL: https://github.com/apache/pinot/pull/15533#discussion_r2042314241
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/GrpcSendingMailbox.java:
##
@@ -131,9 +132,8 @@ public void cancel(Throwable t) {
try {
yashmayya merged PR #15531:
URL: https://github.com/apache/pinot/pull/15531
--
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.ap
This is an automated email from the ASF dual-hosted git repository.
yashmayya 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 1515f49c4c Fix semi-join / pipeline breaker on
yashmayya closed issue #15530: Semi join pipeline breaker fails with types like
BYTES and BIG_DECIMAL in the multi-stage engine
URL: https://github.com/apache/pinot/issues/15530
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
gortiz commented on code in PR #15533:
URL: https://github.com/apache/pinot/pull/15533#discussion_r2042148147
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/QueryErrorCode.java:
##
@@ -57,6 +57,8 @@ public enum QueryErrorCode {
UNKNOWN_COLUMN(710, "UnknownColumnErr
codecov-commenter commented on PR #15538:
URL: https://github.com/apache/pinot/pull/15538#issuecomment-2801835826
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15538?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
This is an automated email from the ASF dual-hosted git repository.
gortiz pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from ecc367c797 Add null handling support for MV aggregation functions -
COUNT, MIN, MAX, SUM, AVG, MINMAXRANGE (#15524)
gortiz merged PR #15534:
URL: https://github.com/apache/pinot/pull/15534
--
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.apach
gortiz commented on code in PR #15533:
URL: https://github.com/apache/pinot/pull/15533#discussion_r2042154801
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/QueryErrorCode.java:
##
@@ -153,4 +155,14 @@ public boolean isClientError() {
return false;
}
gortiz commented on code in PR #15533:
URL: https://github.com/apache/pinot/pull/15533#discussion_r2042140226
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/GrpcSendingMailbox.java:
##
@@ -131,9 +132,8 @@ public void cancel(Throwable t) {
try {
S
codecov-commenter commented on PR #15536:
URL: https://github.com/apache/pinot/pull/15536#issuecomment-2801501718
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15536?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
gortiz opened a new pull request, #15534:
URL: https://github.com/apache/pinot/pull/15534
This PR removes a problematic check when inferPartitionHint is true. The
original idea was to fail when the worker manager is null in a broker and this
query option is true. However, this situation is
codecov-commenter commented on PR #15535:
URL: https://github.com/apache/pinot/pull/15535#issuecomment-2801497355
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15535?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
dependabot[bot] opened a new pull request, #15536:
URL: https://github.com/apache/pinot/pull/15536
Bumps commons-io:commons-io from 2.11.0 to 2.19.0.
[ under o
codecov-commenter commented on PR #15533:
URL: https://github.com/apache/pinot/pull/15533#issuecomment-2801267240
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15533?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
abhishekbafna commented on code in PR #15515:
URL: https://github.com/apache/pinot/pull/15515#discussion_r2041813679
##
pinot-common/src/main/java/org/apache/pinot/common/utils/LogicalTableUtils.java:
##
@@ -0,0 +1,66 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF)
albertobastos closed pull request #15532: Reduce logging spam
URL: https://github.com/apache/pinot/pull/15532
--
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-
albertobastos opened a new pull request, #15533:
URL: https://github.com/apache/pinot/pull/15533
This PR aims to reduce how we spam the logs under certain query error
scenarios, usually including big stack traces that are either irrelevant or
redundant.
Main changes include:
-
codecov-commenter commented on PR #15532:
URL: https://github.com/apache/pinot/pull/15532#issuecomment-2800963211
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15532?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
codecov-commenter commented on PR #15531:
URL: https://github.com/apache/pinot/pull/15531#issuecomment-2800931991
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15531?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
yashmayya merged PR #15524:
URL: https://github.com/apache/pinot/pull/15524
--
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.ap
This is an automated email from the ASF dual-hosted git repository.
yashmayya pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 8af86e0783 Fix MIN_MAX_RANGE aggregation function behavior when all
input values are null (#15525)
add ecc367c
This is an automated email from the ASF dual-hosted git repository.
gortiz pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 883d8ee36a Substitute TransferableBlock with MseBlock (#15245)
add 8af86e0783 Fix MIN_MAX_RANGE aggregation functi
gortiz merged PR #15525:
URL: https://github.com/apache/pinot/pull/15525
--
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.apach
gortiz commented on code in PR #15203:
URL: https://github.com/apache/pinot/pull/15203#discussion_r2041666301
##
pinot-broker/src/main/java/org/apache/pinot/broker/routing/adaptiveserverselector/PriorityGroupInstanceSelector.java:
##
@@ -0,0 +1,194 @@
+/**
+ * Licensed to the Ap
albertobastos opened a new pull request, #15532:
URL: https://github.com/apache/pinot/pull/15532
This PR aims to reduce how we spam the logs under certain query error
scenarios, usually including big stack traces that are either irrelevant or
redundant.
Main changes include:
-
gortiz commented on PR #15335:
URL: https://github.com/apache/pinot/pull/15335#issuecomment-2800825514
> Thanks @gortiz for pointing this out, we can tune this memory allocated to
direct buffers. On that note, I am thinking to assign 1/5th of unused offheap
mem instead of 2/3rd of the total
INNOCENT-BOY closed issue #12094: Pinot Task Framework on Server Nodes
URL: https://github.com/apache/pinot/issues/12094
--
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 uns
yashmayya opened a new issue, #15530:
URL: https://github.com/apache/pinot/issues/15530
- Sample quickstart queries and associated errors:
- `select * from billing where baseUsage IN (select baseUsage from billing
where creditHistory = 'A');`
- `java.lang.IllegalStateException: I
86 matches
Mail list logo
