[PR] Include stats on cancel [pinot]

2025-04-21 Thread via GitHub
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

[I] Kinesis partition split fixes [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Adding segment level QueryCache for expensive predicates [pinot]

2025-04-21 Thread via GitHub
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

[I] Support query options in logical table config [pinot]

2025-04-21 Thread via GitHub
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

[PR] Adding segment level QueryCache for expensive predicates [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Add hooks to service starters to apply custom cluster and instance configs [pinot]

2025-04-21 Thread via GitHub
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)

Re: [PR] Preserve the Order of the Pre-check Items [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Dynamic Buffer Sizing for Dictionary Forward Indexes [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-21 Thread via GitHub
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

Re: [I] JSON_EXTRACT_INDEX null pointer exception, this._arguments is null [pinot]

2025-04-21 Thread via GitHub
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

Re: [I] Pinot-Server keep on going to "CrashLoopBackOff" error [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] roundDecimal() fails for null FLOAT values [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] [Minor] Make all constants in ControllerConf public access [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Metrics for Segment Upload/Download [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch master updated: [Minor] Make all constants in public access (#15605)

2025-04-21 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/pinot.git The following commit(s) were added to refs/heads/master by this push: new 1fdeadd6ef [Minor] Make all constants in public

Re: [PR] [Minor] Make all constants in ControllerConf public access [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Pass Table Config to timeSeries Plugin to use in logical plan phase. [pinot]

2025-04-21 Thread via GitHub
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

[PR] [Minor] Make all constants in public access [pinot]

2025-04-21 Thread via GitHub
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,

(pinot) branch master updated: Add hooks to service starters to apply custom cluster and instance configs (#15595)

2025-04-21 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/pinot.git The following commit(s) were added to refs/heads/master by this push: new 330f0a511c Add hooks to service starters to apply

Re: [PR] Add hooks to service starters to apply custom cluster and instance configs [pinot]

2025-04-21 Thread via GitHub
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

[PR] Pass Table Config to timeSeries Plugin to use in logical plan phase. [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch master updated: Enhance ClusterTest.sanitizeResponse() to reduce error (#15593)

2025-04-21 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/pinot.git The following commit(s) were added to refs/heads/master by this push: new c5c4f7d0c4 Enhance ClusterTest.sanitizeResponse()

Re: [PR] Enhance ClusterTest.sanitizeResponse() to reduce error [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] [Priority Replica Group Routing] Support orderedPreferredReplicas query option for customizable routing strategy [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Add hooks to service starters to apply custom cluster and instance configs [pinot]

2025-04-21 Thread via GitHub
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

Re: [I] Automate Pinot release validation process [pinot]

2025-04-21 Thread via GitHub
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

Re: [I] Introducing more StateModel with different priority [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch master updated: Allow controller to skip PinotFS setup test during boot (#15358)

2025-04-21 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/pinot.git The following commit(s) were added to refs/heads/master by this push: new 8e1c206763 Allow controller to skip PinotFS setup

Re: [PR] Allow controller to skip PinotFS setup test during boot [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Fix filtered aggregate expression hint override logic for compound filters [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch master updated: Fix filtered aggregate expression hint override logic for compound filters (#15603)

2025-04-21 Thread yashmayya
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

Re: [PR] Upgrade Calcite to 1.39.0 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Upgrade Calcite to 1.39.0 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Upgrade Calcite to 1.39.0 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Upgrade Calcite to 1.39.0 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Upgrade Calcite to 1.39.0 [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch master updated: Normalize excessive whitespaces in sql to avoid regex performance issues (#15498)

2025-04-21 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/pinot.git The following commit(s) were added to refs/heads/master by this push: new 290914c4f2 Normalize excessive whitespaces in sql

Re: [PR] Normalize excessive whitespaces in sql to avoid regex performance issues [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.31.25 deleted (was e975cdeef9)

2025-04-21 Thread yashmayya
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

(pinot) branch master updated (693c20f790 -> f006619e91)

2025-04-21 Thread yashmayya
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

Re: [PR] Bump software.amazon.awssdk:bom from 2.31.22 to 2.31.25 [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch dependabot/maven/com.google.errorprone-error_prone_annotations-2.38.0 deleted (was 89d4beb833)

2025-04-21 Thread yashmayya
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

(pinot) branch dependabot/maven/info.picocli-picocli-4.7.7 deleted (was fa2b59f97d)

2025-04-21 Thread yashmayya
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

(pinot) branch master updated (5c9aea6d0b -> 693c20f790)

2025-04-21 Thread yashmayya
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

Re: [PR] Bump com.google.errorprone:error_prone_annotations from 2.37.0 to 2.38.0 [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch master updated (2d8d8c3a4d -> 5c9aea6d0b)

2025-04-21 Thread yashmayya
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

Re: [PR] Bump info.picocli:picocli from 4.7.6 to 4.7.7 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Fix filtered aggregate expression hint override logic for compound filters [pinot]

2025-04-21 Thread via GitHub
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

[PR] Fix filtered aggregate expression hint override logic for compound filters [pinot]

2025-04-21 Thread via GitHub
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.

Re: [PR] Bump software.amazon.awssdk:bom from 2.31.22 to 2.31.25 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Bump com.google.errorprone:error_prone_annotations from 2.37.0 to 2.38.0 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Bump info.picocli:picocli from 4.7.6 to 4.7.7 [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.31.25 created (now e975cdeef9)

2025-04-21 Thread github-bot
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

Re: [PR] Bump software.amazon.awssdk:bom from 2.31.22 to 2.31.24 [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.31.24 deleted (was 877203a802)

2025-04-21 Thread github-bot
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

Re: [PR] Bump software.amazon.awssdk:bom from 2.31.22 to 2.31.24 [pinot]

2025-04-21 Thread via GitHub
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

[PR] Bump software.amazon.awssdk:bom from 2.31.22 to 2.31.25 [pinot]

2025-04-21 Thread via GitHub
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 | | --- | --- |

Re: [PR] Make variables and a method protected to extend from DefaultTableDataManagerProvider & BaseTableDataManager [pinot]

2025-04-21 Thread via GitHub
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

[I] Enhancement: Automated reviews with coderabbit.ai [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch dependabot/maven/com.google.errorprone-error_prone_annotations-2.38.0 created (now 89d4beb833)

2025-04-21 Thread github-bot
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

[PR] Bump com.google.errorprone:error_prone_annotations from 2.37.0 to 2.38.0 [pinot]

2025-04-21 Thread via GitHub
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

[PR] Bump info.picocli:picocli from 4.7.6 to 4.7.7 [pinot]

2025-04-21 Thread via GitHub
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

(pinot) branch dependabot/maven/info.picocli-picocli-4.7.7 created (now fa2b59f97d)

2025-04-21 Thread github-bot
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

Re: [PR] Kinesis partition split fixes [pinot]

2025-04-21 Thread via GitHub
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

[PR] Make variables and a method protected to extend from DefaultTableDataManagerProvider & BaseTableDataManager [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] byte-based splitting for mse query blocks [pinot]

2025-04-21 Thread via GitHub
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)

(pinot) branch master updated (4442d4199e -> 2d8d8c3a4d)

2025-04-21 Thread gortiz
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

Re: [I] [Flaky-test] MailboxServiceTest.testRemoteEarlyTerminated [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Fix 15580 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] Fix 15580 [pinot]

2025-04-21 Thread via GitHub
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

Re: [PR] byte-based splitting for mse query blocks [pinot]

2025-04-21 Thread via GitHub
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)

Re: [PR] byte-based splitting for mse query blocks [pinot]

2025-04-21 Thread via GitHub
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)

Re: [PR] byte-based splitting for mse query blocks [pinot]

2025-04-21 Thread via GitHub
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)

Re: [PR] Reduce segment metadata memory overhead. [pinot]

2025-04-21 Thread via GitHub
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