gortiz opened a new pull request, #15609:
URL: https://github.com/apache/pinot/pull/15609
This PR adds the ability to include stats on timeouts. Although we are ready
to return stats on errors (since https://github.com/apache/pinot/pull/15245),
the tricky part on timeouts is that different
krishan1390 opened a new issue, #15608:
URL: https://github.com/apache/pinot/issues/15608
Kinesis provides capabilities to split a shard into 2 shards and merge 2
shards into 1 shard.
Both these operations create new shards and the older shards (called parent
shards) stop being acti
codecov-commenter commented on PR #15606:
URL: https://github.com/apache/pinot/pull/15606#issuecomment-2820099649
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15606?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
vrajat opened a new issue, #15607:
URL: https://github.com/apache/pinot/issues/15607
Table config supports a few configurations to be applied at query runtime.
These have been documented in
https://docs.pinot.apache.org/configuration-reference/table#query
Support all these configura
xiangfu0 opened a new pull request, #15606:
URL: https://github.com/apache/pinot/pull/15606
(no comment)
--
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-m
yashmayya commented on code in PR #15595:
URL: https://github.com/apache/pinot/pull/15595#discussion_r2053264556
##
pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java:
##
@@ -162,10 +162,10 @@ public void init(PinotConfiguration brokerConf)
Jackie-Jiang commented on code in PR #15496:
URL: https://github.com/apache/pinot/pull/15496#discussion_r2053103644
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java:
##
@@ -77,7 +78,9 @@ public Map
check(PreCheck
Jackie-Jiang commented on code in PR #15407:
URL: https://github.com/apache/pinot/pull/15407#discussion_r2053102277
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/impl/forward/FixedByteSVMutableForwardIndex.java:
##
@@ -84,6 +100,19 @@ public FixedB
Jackie-Jiang commented on PR #15523:
URL: https://github.com/apache/pinot/pull/15523#issuecomment-2819693260
@gortiz @yashmayya Can you help review 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
Jackie-Jiang commented on issue #15516:
URL: https://github.com/apache/pinot/issues/15516#issuecomment-2819691113
@gortiz @yashmayya Can you help take a look?
--
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
Jackie-Jiang commented on issue #15540:
URL: https://github.com/apache/pinot/issues/15540#issuecomment-2819688492
Are you able to find more details from Helix log or Kubernetes log?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
Jackie-Jiang commented on code in PR #15377:
URL: https://github.com/apache/pinot/pull/15377#discussion_r2053095073
##
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/RoundDecimalTransformFunction.java:
##
@@ -90,18 +90,33 @@ public double[] transformT
codecov-commenter commented on PR #15605:
URL: https://github.com/apache/pinot/pull/15605#issuecomment-2819587233
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15605?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
praveenc7 commented on PR #15585:
URL: https://github.com/apache/pinot/pull/15585#issuecomment-2819557580
@dinoocch @vvivekiyer When you get a chance to review this
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
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 1fdeadd6ef [Minor] Make all constants in public
Jackie-Jiang merged PR #15605:
URL: https://github.com/apache/pinot/pull/15605
--
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
codecov-commenter commented on PR #15604:
URL: https://github.com/apache/pinot/pull/15604#issuecomment-2819532093
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15604?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, #15605:
URL: https://github.com/apache/pinot/pull/15605
(no comment)
--
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,
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 330f0a511c Add hooks to service starters to apply
Jackie-Jiang merged PR #15595:
URL: https://github.com/apache/pinot/pull/15595
--
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
raghavyadav01 opened a new pull request, #15604:
URL: https://github.com/apache/pinot/pull/15604
Commit contains following changes
- Pass Table Configs to TSE query Plugin.
- Table Config will be used build logical plan based on schema.
--
This is an automated message from the A
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 c5c4f7d0c4 Enhance ClusterTest.sanitizeResponse()
Jackie-Jiang merged PR #15593:
URL: https://github.com/apache/pinot/pull/15593
--
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
wirybeaver commented on PR #15203:
URL: https://github.com/apache/pinot/pull/15203#issuecomment-2819368086
@gortiz @chenboat @ankitsultana Could you review the PR in another round? I
have resolved the comments
--
This is an automated message from the Apache Git Service.
To respond to the
Jackie-Jiang commented on code in PR #15595:
URL: https://github.com/apache/pinot/pull/15595#discussion_r2052861109
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/util/HelixSetupUtils.java:
##
@@ -60,39 +61,56 @@ private HelixSetupUtils() {
private
abhioncbr commented on issue #15459:
URL: https://github.com/apache/pinot/issues/15459#issuecomment-2819246575
Sure. Please go ahaed
--
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 c
Jackie-Jiang commented on issue #15056:
URL: https://github.com/apache/pinot/issues/15056#issuecomment-2819215889
@itschrispeck This applies only when server is starting. After server
starts, if we handle CONSUMING in the end, it could potentially make things
worse because CONSUMING transit
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 8e1c206763 Allow controller to skip PinotFS setup
Jackie-Jiang merged PR #15358:
URL: https://github.com/apache/pinot/pull/15358
--
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
yashmayya merged PR #15603:
URL: https://github.com/apache/pinot/pull/15603
--
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 936e9c3115 Fix filtered aggregate expression h
Jackie-Jiang commented on code in PR #15263:
URL: https://github.com/apache/pinot/pull/15263#discussion_r2052787843
##
pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/rules/PinotEvaluateLiteralRule.java:
##
@@ -258,13 +275,18 @@ private static Object convertResult
yashmayya commented on code in PR #15263:
URL: https://github.com/apache/pinot/pull/15263#discussion_r2052756575
##
pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/rules/PinotEvaluateLiteralRule.java:
##
@@ -258,13 +275,18 @@ private static Object convertResultVal
yashmayya commented on code in PR #15263:
URL: https://github.com/apache/pinot/pull/15263#discussion_r2052753412
##
pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/rules/PinotEvaluateLiteralRule.java:
##
@@ -258,13 +275,18 @@ private static Object convertResultVal
yashmayya commented on code in PR #15263:
URL: https://github.com/apache/pinot/pull/15263#discussion_r2052751502
##
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/MultiStageEngineIntegrationTest.java:
##
@@ -971,7 +971,7 @@ Object[][]
polymorphicScalar
Jackie-Jiang commented on code in PR #15263:
URL: https://github.com/apache/pinot/pull/15263#discussion_r2052738566
##
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/MultiStageEngineIntegrationTest.java:
##
@@ -971,7 +971,7 @@ Object[][]
polymorphicSca
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 290914c4f2 Normalize excessive whitespaces in sql
Jackie-Jiang merged PR #15498:
URL: https://github.com/apache/pinot/pull/15498
--
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.
yashmayya pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.31.25
in repository https://gitbox.apache.org/repos/asf/pinot.git
was e975cdeef9 Bump software.amazon.awssdk:bom from 2.31.22 to 2.31.25
Th
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 693c20f790 Bump com.google.errorprone:error_prone_annotations from
2.37.0 to 2.38.0 (#15600)
add f006619e91 Bu
yashmayya merged PR #15602:
URL: https://github.com/apache/pinot/pull/15602
--
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
dependabot/maven/com.google.errorprone-error_prone_annotations-2.38.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 89d4beb833 Bump com.google.errorprone:error_prone_an
This is an automated email from the ASF dual-hosted git repository.
yashmayya pushed a change to branch dependabot/maven/info.picocli-picocli-4.7.7
in repository https://gitbox.apache.org/repos/asf/pinot.git
was fa2b59f97d Bump info.picocli:picocli from 4.7.6 to 4.7.7
The revisions that we
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 5c9aea6d0b Bump info.picocli:picocli from 4.7.6 to 4.7.7 (#15599)
add 693c20f790 Bump com.google.errorprone:err
yashmayya merged PR #15600:
URL: https://github.com/apache/pinot/pull/15600
--
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 2d8d8c3a4d Fix 15580 (#15597)
add 5c9aea6d0b Bump info.picocli:picocli from 4.7.6 to 4.7.7 (#15599)
No new rev
yashmayya merged PR #15599:
URL: https://github.com/apache/pinot/pull/15599
--
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
codecov-commenter commented on PR #15603:
URL: https://github.com/apache/pinot/pull/15603#issuecomment-2818866724
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15603?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, #15603:
URL: https://github.com/apache/pinot/pull/15603
- Currently, a filtered aggregate with a compound filter like `SUM(x) FILTER
(WHERE y > 100 AND y < 100)` fails with an NPE like:
```
java.lang.NullPointerException: Cannot invoke
"org.apache.
codecov-commenter commented on PR #15602:
URL: https://github.com/apache/pinot/pull/15602#issuecomment-2818450644
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15602?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 #15600:
URL: https://github.com/apache/pinot/pull/15600#issuecomment-2818413273
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15600?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 #15599:
URL: https://github.com/apache/pinot/pull/15599#issuecomment-2818381159
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15599?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.
github-bot pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.31.25
in repository https://gitbox.apache.org/repos/asf/pinot.git
at e975cdeef9 Bump software.amazon.awssdk:bom from 2.31.22 to 2.31.25
N
dependabot[bot] closed pull request #15590: Bump software.amazon.awssdk:bom
from 2.31.22 to 2.31.24
URL: https://github.com/apache/pinot/pull/15590
--
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 th
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.31.24
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 877203a802 Bump software.amazon.awssdk:bom from 2.31.22 to 2.31.24
T
dependabot[bot] commented on PR #15590:
URL: https://github.com/apache/pinot/pull/15590#issuecomment-2818378092
Superseded by #15602.
--
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
dependabot[bot] opened a new pull request, #15602:
URL: https://github.com/apache/pinot/pull/15602
Bumps software.amazon.awssdk:bom from 2.31.22 to 2.31.25.
Most Recent Ignore Conditions Applied to This Pull Request
| Dependency Name | Ignore Conditions |
| --- | --- |
codecov-commenter commented on PR #15598:
URL: https://github.com/apache/pinot/pull/15598#issuecomment-2818355206
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15598?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
reneleonhardt opened a new issue, #15601:
URL: https://github.com/apache/pinot/issues/15601
### Improved review process
Allow more community contributions and a faster review process by enabling
CodeRabbit (free for Open-Source repositories).
1. Learn how it works https://coderabbit.ai
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/com.google.errorprone-error_prone_annotations-2.38.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 89d4beb833 Bump com.google.errorprone:error_prone_a
dependabot[bot] opened a new pull request, #15600:
URL: https://github.com/apache/pinot/pull/15600
Bumps
[com.google.errorprone:error_prone_annotations](https://github.com/google/error-prone)
from 2.37.0 to 2.38.0.
Release notes
Sourced from https://github.com/google/error-prone/r
dependabot[bot] opened a new pull request, #15599:
URL: https://github.com/apache/pinot/pull/15599
Bumps [info.picocli:picocli](https://github.com/remkop/picocli) from 4.7.6
to 4.7.7.
Release notes
Sourced from https://github.com/remkop/picocli/releases";>info.picocli:picocli's
re
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/info.picocli-picocli-4.7.7
in repository https://gitbox.apache.org/repos/asf/pinot.git
at fa2b59f97d Bump info.picocli:picocli from 4.7.6 to 4.7.7
No new revisions wer
krishan1390 commented on code in PR #15563:
URL: https://github.com/apache/pinot/pull/15563#discussion_r2052300098
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PauseStatusDetails.java:
##
@@ -65,4 +66,11 @@ public String getComment() {
public St
deepthi912 opened a new pull request, #15598:
URL: https://github.com/apache/pinot/pull/15598
To extend, expose certain variables in TableDataManager classes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
gortiz commented on code in PR #15571:
URL: https://github.com/apache/pinot/pull/15571#discussion_r2052080756
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlockUtils.java:
##
@@ -230,6 +230,28 @@ public static ByteString toByteString(DataBlock dataBlock)
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 4442d4199e Fix range index on raw TIMESTAMP column (#15589)
add 2d8d8c3a4d Fix 15580 (#15597)
No new revisions we
gortiz closed issue #15580: [Flaky-test]
MailboxServiceTest.testRemoteEarlyTerminated
URL: https://github.com/apache/pinot/issues/15580
--
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 c
gortiz merged PR #15597:
URL: https://github.com/apache/pinot/pull/15597
--
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
codecov-commenter commented on PR #15597:
URL: https://github.com/apache/pinot/pull/15597#issuecomment-2817851240
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15597?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
gortiz commented on code in PR #15571:
URL: https://github.com/apache/pinot/pull/15571#discussion_r2052078798
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlockUtils.java:
##
@@ -230,6 +230,28 @@ public static ByteString toByteString(DataBlock dataBlock)
gortiz commented on code in PR #15571:
URL: https://github.com/apache/pinot/pull/15571#discussion_r2052078798
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlockUtils.java:
##
@@ -230,6 +230,28 @@ public static ByteString toByteString(DataBlock dataBlock)
gortiz commented on code in PR #15571:
URL: https://github.com/apache/pinot/pull/15571#discussion_r2052075968
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlockUtils.java:
##
@@ -230,6 +230,28 @@ public static ByteString toByteString(DataBlock dataBlock)
gortiz commented on code in PR #15591:
URL: https://github.com/apache/pinot/pull/15591#discussion_r2052068409
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/metadata/ColumnMetadataImpl.java:
##
@@ -164,10 +172,40 @@ public Set getPartitions() {
return
74 matches
Mail list logo