Re: [PR] [MSQE] Add a plan listener to report rule optimizer evaluation time [pinot]

2025-03-05 Thread via GitHub
gortiz commented on code in PR #15192: URL: https://github.com/apache/pinot/pull/15192#discussion_r1982856026 ## pinot-query-planner/src/main/java/org/apache/pinot/query/context/PlannerContext.java: ## @@ -49,6 +50,7 @@ public class PlannerContext implements AutoCloseable { p

Re: [PR] Broker list fetching API in PinotConnection is fetching dead servers … [pinot]

2025-03-05 Thread via GitHub
codecov-commenter commented on PR #15207: URL: https://github.com/apache/pinot/pull/15207#issuecomment-2703028207 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15207?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

[PR] Broker list fetching API in PinotConnection is fetching dead servers … [pinot]

2025-03-05 Thread via GitHub
chrajeshbabu opened a new pull request, #15207: URL: https://github.com/apache/pinot/pull/15207 …as well which leads to query failures sometimes (#15187) -- 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: [PR] Skip logs for virtual columns as they are known not to be in the schema [pinot]

2025-03-05 Thread via GitHub
mayankshriv commented on code in PR #15206: URL: https://github.com/apache/pinot/pull/15206#discussion_r1982754431 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/loader/ForwardIndexHandler.java: ## @@ -204,7 +205,11 @@ Map> computeOperations(S

Re: [PR] Fix memory leak in QueryDispatcher [pinot]

2025-03-05 Thread via GitHub
yashmayya merged PR #15201: URL: https://github.com/apache/pinot/pull/15201 -- 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 memory leak in QueryDispatcher (#15201)

2025-03-05 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 d9ffa99285 Fix memory leak in QueryDispatcher

Re: [PR] Fix memory leak in QueryDispatcher [pinot]

2025-03-05 Thread via GitHub
yashmayya commented on code in PR #15201: URL: https://github.com/apache/pinot/pull/15201#discussion_r1982622318 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/service/dispatch/QueryDispatcher.java: ## @@ -146,7 +146,13 @@ public QueryResult submitAndReduce(RequestC

Re: [PR] Enable force overwrite for partial upserts [pinot]

2025-03-05 Thread via GitHub
himanish-star commented on code in PR #15107: URL: https://github.com/apache/pinot/pull/15107#discussion_r1982473423 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/merger/PartialUpsertColumnarMerger.java: ## @@ -74,7 +75,10 @@ public void merge(LazyRo

Re: [PR] [MSQE] Add a plan listener to report rule optimizer evaluation time [pinot]

2025-03-05 Thread via GitHub
xiangfu0 commented on code in PR #15192: URL: https://github.com/apache/pinot/pull/15192#discussion_r1982446491 ## pinot-query-planner/src/main/java/org/apache/pinot/query/context/PlannerContext.java: ## @@ -49,6 +50,7 @@ public class PlannerContext implements AutoCloseable {

Re: [PR] [MSQE] Add a plan listener to report rule optimizer evaluation time [pinot]

2025-03-05 Thread via GitHub
xiangfu0 commented on code in PR #15192: URL: https://github.com/apache/pinot/pull/15192#discussion_r1982446491 ## pinot-query-planner/src/main/java/org/apache/pinot/query/context/PlannerContext.java: ## @@ -49,6 +50,7 @@ public class PlannerContext implements AutoCloseable {

Re: [PR] Add off-heap distinct set implementations [pinot]

2025-03-05 Thread via GitHub
codecov-commenter commented on PR #15205: URL: https://github.com/apache/pinot/pull/15205#issuecomment-2702467486 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15205?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

Re: [PR] Enable force overwrite for partial upserts [pinot]

2025-03-05 Thread via GitHub
klsince commented on code in PR #15107: URL: https://github.com/apache/pinot/pull/15107#discussion_r1982399364 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/merger/PartialUpsertColumnarMerger.java: ## @@ -74,7 +75,10 @@ public void merge(LazyRow prev

[PR] Add off-heap distinct set implementations [pinot]

2025-03-05 Thread via GitHub
Jackie-Jiang opened a new pull request, #15205: URL: https://github.com/apache/pinot/pull/15205 Adds distinct set implementations that can be used for accurate distinct count: - 32 bit - 64 bit - 128 bit -- This is an automated message from the Apache Git Service. To respond to t

Re: [PR] Fix memory leak in QueryDispatcher [pinot]

2025-03-05 Thread via GitHub
ankitsultana commented on code in PR #15201: URL: https://github.com/apache/pinot/pull/15201#discussion_r1982340395 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/service/dispatch/QueryDispatcher.java: ## @@ -146,7 +146,13 @@ public QueryResult submitAndReduce(Reque

Re: [PR] [MSQE] Add a plan listener to report rule optimizer evaluation time [pinot]

2025-03-05 Thread via GitHub
Jackie-Jiang commented on code in PR #15192: URL: https://github.com/apache/pinot/pull/15192#discussion_r1982308252 ## pinot-query-planner/src/main/java/org/apache/pinot/query/context/PlannerContext.java: ## @@ -49,6 +50,7 @@ public class PlannerContext implements AutoCloseable

Re: [PR] Add support for orphan segment cleanup [pinot]

2025-03-05 Thread via GitHub
klsince commented on code in PR #15142: URL: https://github.com/apache/pinot/pull/15142#discussion_r1982294881 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/retention/RetentionManager.java: ## @@ -77,6 +88,7 @@ protected void processTable(String tableN

Re: [I] `ArrayIndexOutOfBoundsException` error for multi-stage query [pinot]

2025-03-05 Thread via GitHub
dang-stripe commented on issue #15121: URL: https://github.com/apache/pinot/issues/15121#issuecomment-2702270791 Here's the smaller version of the query that reproduces it. The window function isn't necessary. ``` WITH "simple_data" AS ( SELECT "user_id", "value",

Re: [I] `ArrayIndexOutOfBoundsException` error for multi-stage query [pinot]

2025-03-05 Thread via GitHub
Jackie-Jiang commented on issue #15121: URL: https://github.com/apache/pinot/issues/15121#issuecomment-2702264622 @yashmayya The issue might be introduced by #14211. I tried putting the option `SET filteredAggregationsSkipEmptyGroups = true;` and the query works. The exception is from `Mult

Re: [PR] Add fetchPartitionCount and fetchStreamPartitionOffset implementation api for pinot-kinesis [pinot]

2025-03-05 Thread via GitHub
xiangfu0 merged PR #14713: URL: https://github.com/apache/pinot/pull/14713 -- 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

(pinot) branch master updated (e43de5e59e -> 07785bf8cf)

2025-03-05 Thread xiangfu
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 e43de5e59e Minion tasks should not pick up problematic consuming segments (#15173) add 07785bf8cf Add fetchParti

Re: [PR] apache#15162 - Bug Fix Segments going into BAD state for Dedup Tables with metadataTTL [pinot]

2025-03-05 Thread via GitHub
deepthi912 commented on code in PR #15178: URL: https://github.com/apache/pinot/pull/15178#discussion_r1982295648 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -892,8 +892,13 @@ public void run() { }

Re: [PR] apache#15162 - Bug Fix Segments going into BAD state for Dedup Tables with metadataTTL [pinot]

2025-03-05 Thread via GitHub
klsince commented on code in PR #15178: URL: https://github.com/apache/pinot/pull/15178#discussion_r1982270429 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -892,8 +892,13 @@ public void run() { } }

(pinot) branch master updated (a5ee332bf7 -> e43de5e59e)

2025-03-05 Thread sajjad
This is an automated email from the ASF dual-hosted git repository. sajjad pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from a5ee332bf7 Code cleanup to reduce nested level. (#15202) add e43de5e59e Minion tasks should not pick up problemati

Re: [PR] Minion tasks should not pick up problematic consuming segments [pinot]

2025-03-05 Thread via GitHub
sajjad-moradi merged PR #15173: URL: https://github.com/apache/pinot/pull/15173 -- 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...@pino

Re: [I] `ArrayIndexOutOfBoundsException` error for multi-stage query [pinot]

2025-03-05 Thread via GitHub
dang-stripe commented on issue #15121: URL: https://github.com/apache/pinot/issues/15121#issuecomment-2702206671 We found the stacktrace after restarting the server: ``` [2025-03-05 22:17:19.006030] java.lang.ArrayIndexOutOfBoundsException: Index 232 out of bounds for length 200 [20

Re: [PR] apache#15162 - Bug Fix Segments going into BAD state for Dedup Tables with metadataTTL [pinot]

2025-03-05 Thread via GitHub
deepthi912 commented on code in PR #15178: URL: https://github.com/apache/pinot/pull/15178#discussion_r1982241163 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/dedup/BasePartitionDedupMetadataManager.java: ## @@ -283,8 +283,13 @@ public void removeSegment(I

Re: [PR] apache#15162 - Bug Fix Segments going into BAD state for Dedup Tables with metadataTTL [pinot]

2025-03-05 Thread via GitHub
deepthi912 commented on code in PR #15178: URL: https://github.com/apache/pinot/pull/15178#discussion_r1982241163 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/dedup/BasePartitionDedupMetadataManager.java: ## @@ -283,8 +283,13 @@ public void removeSegment(I

Re: [PR] apache#15162 - Bug Fix Segments going into BAD state for Dedup Tables with metadataTTL [pinot]

2025-03-05 Thread via GitHub
deepthi912 commented on code in PR #15178: URL: https://github.com/apache/pinot/pull/15178#discussion_r1982241163 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/dedup/BasePartitionDedupMetadataManager.java: ## @@ -283,8 +283,13 @@ public void removeSegment(I

Re: [PR] apache#15162 - Bug Fix Segments going into BAD state for Dedup Tables with metadataTTL [pinot]

2025-03-05 Thread via GitHub
klsince commented on code in PR #15178: URL: https://github.com/apache/pinot/pull/15178#discussion_r1982207064 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/dedup/BasePartitionDedupMetadataManager.java: ## @@ -283,8 +283,13 @@ public void removeSegment(Inde

Re: [PR] SAR-628 - Add Raft blog post [pinot-site]

2025-03-05 Thread via GitHub
xiangfu0 merged PR #143: URL: https://github.com/apache/pinot-site/pull/143 -- 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-site) branch new-site-dev updated: SAR-628 - Add Raft blog post (#143)

2025-03-05 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch new-site-dev in repository https://gitbox.apache.org/repos/asf/pinot-site.git The following commit(s) were added to refs/heads/new-site-dev by this push: new f0c463f2 SAR-628 - Add Raft blo

(pinot) branch master updated (f4c1cac8c1 -> a5ee332bf7)

2025-03-05 Thread lqc
This is an automated email from the ASF dual-hosted git repository. lqc pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from f4c1cac8c1 Fix GET /queries broker API UnsupportedOperationException bug (#15200) add a5ee332bf7 Code cleanup to red

Re: [PR] Code cleanup to reduce nested level. [pinot]

2025-03-05 Thread via GitHub
deemoliu merged PR #15202: URL: https://github.com/apache/pinot/pull/15202 -- 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

[PR] SAR-628 - Add Raft blog post [pinot-site]

2025-03-05 Thread via GitHub
gio-startree opened a new pull request, #143: URL: https://github.com/apache/pinot-site/pull/143 (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

Re: [PR] apache#15162 - Bug Fix Segments going into BAD state for Dedup Tables with metadataTTL [pinot]

2025-03-05 Thread via GitHub
klsince commented on code in PR #15178: URL: https://github.com/apache/pinot/pull/15178#discussion_r1982131547 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/dedup/BasePartitionDedupMetadataManager.java: ## @@ -283,8 +283,13 @@ public void removeSegment(Inde

[I] CalciteSqlParser fails to parse table name from queries with window functions [pinot]

2025-03-05 Thread via GitHub
AlexanderKM opened a new issue, #15204: URL: https://github.com/apache/pinot/issues/15204 We use the `RequestUtils#getTableNames` a bit on our end to track and report which tables our queries are hitting, and this method seems to be failing for queries that contain window functions. Under t

Re: [PR] Add base class for merging and accumulating custom objects [pinot]

2025-03-05 Thread via GitHub
itschrispeck commented on PR #12685: URL: https://github.com/apache/pinot/pull/12685#issuecomment-2701898097 @davecromberge iiuc, the CPC sketch equivalent linked change was already added by you? https://github.com/apache/pinot/pull/12685/files#diff-36a307b118c7aa2dda45445ada55d0c5f

Re: [PR] [AdaptiveServerSelector] pass in the replica group list ordered by preference [pinot]

2025-03-05 Thread via GitHub
codecov-commenter commented on PR #15203: URL: https://github.com/apache/pinot/pull/15203#issuecomment-2701788023 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15203?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

Re: [PR] Code cleanup to reduce nested level. [pinot]

2025-03-05 Thread via GitHub
codecov-commenter commented on PR #15202: URL: https://github.com/apache/pinot/pull/15202#issuecomment-2701735891 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15202?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

[PR] [AdaptiveServerSelector] pass in the replica group list ordered by preference [pinot]

2025-03-05 Thread via GitHub
wirybeaver opened a new pull request, #15203: URL: https://github.com/apache/pinot/pull/15203 (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

[PR] Code cleanup to reduce nested level. [pinot]

2025-03-05 Thread via GitHub
chenboat opened a new pull request, #15202: URL: https://github.com/apache/pinot/pull/15202 Instructions: 1. The PR has to be tagged with at least one of the following labels (*): 1. `feature` 2. `bugfix` 3. `performance` 4. `ui` 5. `backward-incompat` 6

(pinot) branch master updated (8b5766db0d -> 1705ca83a4)

2025-03-05 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 8b5766db0d Bump io.grpc:grpc-bom from 1.70.0 to 1.71.0 (#15196) add 1705ca83a4 Bump testcontainers.version from

Re: [PR] Fix memory leak in QueryDispatcher [pinot]

2025-03-05 Thread via GitHub
codecov-commenter commented on PR #15201: URL: https://github.com/apache/pinot/pull/15201#issuecomment-2701488224 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15201?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

Re: [PR] Fix memory leak in QueryDispatcher [pinot]

2025-03-05 Thread via GitHub
yashmayya commented on code in PR #15201: URL: https://github.com/apache/pinot/pull/15201#discussion_r1981751906 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/service/dispatch/QueryDispatcher.java: ## @@ -146,7 +146,13 @@ public QueryResult submitAndReduce(RequestC

Re: [PR] Fix memory leak in QueryDispatcher [pinot]

2025-03-05 Thread via GitHub
yashmayya commented on code in PR #15201: URL: https://github.com/apache/pinot/pull/15201#discussion_r1981749262 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/service/dispatch/QueryDispatcher.java: ## @@ -146,7 +146,13 @@ public QueryResult submitAndReduce(RequestC

Re: [PR] [MSQE] Add a plan listener to report rule optimizer evaluation time [pinot]

2025-03-05 Thread via GitHub
gortiz commented on code in PR #15192: URL: https://github.com/apache/pinot/pull/15192#discussion_r1981726584 ## pinot-query-planner/src/main/java/org/apache/pinot/query/context/RuleTimingPlannerListener.java: ## @@ -0,0 +1,89 @@ +package org.apache.pinot.query.context; + +/** +

Re: [PR] Fix memory leak in QueryDispatcher [pinot]

2025-03-05 Thread via GitHub
gortiz commented on code in PR #15201: URL: https://github.com/apache/pinot/pull/15201#discussion_r1981735056 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/service/dispatch/QueryDispatcher.java: ## @@ -146,7 +146,13 @@ public QueryResult submitAndReduce(RequestCont

[PR] Fix memory leak in QueryDispatcher [pinot]

2025-03-05 Thread via GitHub
yashmayya opened a new pull request, #15201: URL: https://github.com/apache/pinot/pull/15201 - https://github.com/apache/pinot/commit/f1509f8a4339181d8540abd5e9f66e2d25bb67d3 introduced a `Map> _serversByQuery` in `QueryDispatcher` that tracks the set of servers that a query has been disp

Re: [PR] Fix GET /queries broker API UnsupportedOperationException bug [pinot]

2025-03-05 Thread via GitHub
yashmayya merged PR #15200: URL: https://github.com/apache/pinot/pull/15200 -- 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 GET /queries broker API UnsupportedOperationException bug (#15200)

2025-03-05 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 f4c1cac8c1 Fix GET /queries broker API Unsuppo

Re: [PR] Refactor query error handling to use QueryErrorCode and QueryErrorMessage for improved clarity and consistency [pinot]

2025-03-05 Thread via GitHub
gortiz commented on PR #15037: URL: https://github.com/apache/pinot/pull/15037#issuecomment-2701257236 checks are green again -- 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.

Re: [PR] Fix GET /queries broker API UnsupportedOperationException bug [pinot]

2025-03-05 Thread via GitHub
codecov-commenter commented on PR #15200: URL: https://github.com/apache/pinot/pull/15200#issuecomment-2701244082 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15200?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

(pinot) branch master updated: Bump software.amazon.awssdk:bom from 2.30.32 to 2.30.33 (#15199)

2025-03-05 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 3ffcf28df1 Bump software.amazon.awssdk:bom fro

[PR] Fix GET /queries broker API UnsupportedOperationException bug [pinot]

2025-03-05 Thread via GitHub
yashmayya opened a new pull request, #15200: URL: https://github.com/apache/pinot/pull/15200 - This method was updated in https://github.com/apache/pinot/pull/14823 which introduced a new cancellation mechanism. - A small bug was introduced here as well wherein calls to `GET /queries` fa

(pinot) branch dependabot/maven/testcontainers.version-1.20.6 deleted (was 7a235ae004)

2025-03-05 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch dependabot/maven/testcontainers.version-1.20.6 in repository https://gitbox.apache.org/repos/asf/pinot.git was 7a235ae004 Bump testcontainers.version from 1.20.5 to 1.20.6 The revisions

(pinot) branch master updated (3ffcf28df1 -> 8b5766db0d)

2025-03-05 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 3ffcf28df1 Bump software.amazon.awssdk:bom from 2.30.32 to 2.30.33 (#15199) add 8b5766db0d Bump io.grpc:grpc-b

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.30.33 deleted (was c23b83009c)

2025-03-05 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.30.33 in repository https://gitbox.apache.org/repos/asf/pinot.git was c23b83009c Bump software.amazon.awssdk:bom from 2.30.32 to 2.30.33 Th

Re: [PR] Bump io.grpc:grpc-bom from 1.70.0 to 1.71.0 [pinot]

2025-03-05 Thread via GitHub
yashmayya merged PR #15196: URL: https://github.com/apache/pinot/pull/15196 -- 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/io.grpc-grpc-bom-1.71.0 deleted (was 57644d0c37)

2025-03-05 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch dependabot/maven/io.grpc-grpc-bom-1.71.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was 57644d0c37 Bump io.grpc:grpc-bom from 1.70.0 to 1.71.0 The revisions that were on

Re: [PR] Bump testcontainers.version from 1.20.5 to 1.20.6 [pinot]

2025-03-05 Thread via GitHub
yashmayya merged PR #15198: URL: https://github.com/apache/pinot/pull/15198 -- 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] Bump software.amazon.awssdk:bom from 2.30.32 to 2.30.33 [pinot]

2025-03-05 Thread via GitHub
yashmayya merged PR #15199: URL: https://github.com/apache/pinot/pull/15199 -- 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: [I] Pinot Helm Chart 0.3.1 Release [pinot]

2025-03-05 Thread via GitHub
PovilasV1 commented on issue #15194: URL: https://github.com/apache/pinot/issues/15194#issuecomment-2700861102 cc @abhioncbr @xiangfu0 -- 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 specif

(pinot) branch master updated (f1168918ff -> 504fdca562)

2025-03-05 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 f1168918ff Relocate fmpp to maven-antrun-plugin (#15108) add 504fdca562 fix MSE incorrect stats visualization (#15

Re: [PR] fix MSE incorrect stats visualization [pinot]

2025-03-05 Thread via GitHub
gortiz merged PR #15188: URL: https://github.com/apache/pinot/pull/15188 -- 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] Bump com.puppycrawl.tools:checkstyle from 10.21.3 to 10.21.4 [pinot]

2025-03-05 Thread via GitHub
Jackie-Jiang merged PR #15182: URL: https://github.com/apache/pinot/pull/15182 -- 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] Bump com.thoughtworks.paranamer:paranamer from 2.8 to 2.8.2 [pinot]

2025-03-05 Thread via GitHub
dependabot[bot] opened a new pull request, #15197: URL: https://github.com/apache/pinot/pull/15197 Bumps [com.thoughtworks.paranamer:paranamer](https://github.com/paul-hammant/paranamer) from 2.8 to 2.8.2. Commits https://github.com/paul-hammant/paranamer/commit/39ac5ed8109302

[PR] Bump testcontainers.version from 1.20.5 to 1.20.6 [pinot]

2025-03-05 Thread via GitHub
dependabot[bot] opened a new pull request, #15198: URL: https://github.com/apache/pinot/pull/15198 Bumps `testcontainers.version` from 1.20.5 to 1.20.6. Updates `org.testcontainers:testcontainers` from 1.20.5 to 1.20.6 Release notes Sourced from https://github.com/testcontainers/

(pinot) branch dependabot/maven/testcontainers.version-1.20.6 created (now 7a235ae004)

2025-03-05 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/testcontainers.version-1.20.6 in repository https://gitbox.apache.org/repos/asf/pinot.git at 7a235ae004 Bump testcontainers.version from 1.20.5 to 1.20.6 No new revis

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.30.33 created (now c23b83009c)

2025-03-05 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.30.33 in repository https://gitbox.apache.org/repos/asf/pinot.git at c23b83009c Bump software.amazon.awssdk:bom from 2.30.32 to 2.30.33 N

[PR] Bump software.amazon.awssdk:bom from 2.30.32 to 2.30.33 [pinot]

2025-03-05 Thread via GitHub
dependabot[bot] opened a new pull request, #15199: URL: https://github.com/apache/pinot/pull/15199 Bumps software.amazon.awssdk:bom from 2.30.32 to 2.30.33. Most Recent Ignore Conditions Applied to This Pull Request | Dependency Name | Ignore Conditions | | --- | --- |

(pinot) branch dependabot/maven/com.thoughtworks.paranamer-paranamer-2.8.2 created (now 65b4744995)

2025-03-05 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.thoughtworks.paranamer-paranamer-2.8.2 in repository https://gitbox.apache.org/repos/asf/pinot.git at 65b4744995 Bump com.thoughtworks.paranamer:paranamer from 2.8

(pinot) branch dependabot/maven/io.grpc-grpc-bom-1.71.0 created (now 57644d0c37)

2025-03-05 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/io.grpc-grpc-bom-1.71.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at 57644d0c37 Bump io.grpc:grpc-bom from 1.70.0 to 1.71.0 No new revisions were add

[PR] Bump io.grpc:grpc-bom from 1.70.0 to 1.71.0 [pinot]

2025-03-05 Thread via GitHub
dependabot[bot] opened a new pull request, #15196: URL: https://github.com/apache/pinot/pull/15196 Bumps [io.grpc:grpc-bom](https://github.com/grpc/grpc-java) from 1.70.0 to 1.71.0. Commits https://github.com/grpc/grpc-java/commit/865c4432569dfdf738b3cf07bcdb6d6e3285f761";>865c

Re: [PR] Refactor query error handling to use QueryErrorCode and QueryErrorMessage for improved clarity and consistency [pinot]

2025-03-05 Thread via GitHub
gortiz commented on code in PR #15037: URL: https://github.com/apache/pinot/pull/15037#discussion_r1981212822 ## pinot-common/src/main/java/org/apache/pinot/common/response/BrokerResponse.java: ## @@ -76,11 +76,12 @@ default void toOutputStream(OutputStream outputStream) /**

Re: [PR] Refactor query error handling to use QueryErrorCode and QueryErrorMessage for improved clarity and consistency [pinot]

2025-03-05 Thread via GitHub
gortiz commented on code in PR #15037: URL: https://github.com/apache/pinot/pull/15037#discussion_r1981195226 ## pinot-common/src/main/java/org/apache/pinot/common/response/BrokerResponse.java: ## @@ -76,11 +76,12 @@ default void toOutputStream(OutputStream outputStream) /**

Re: [PR] fix MSE incorrect stats visualization [pinot]

2025-03-05 Thread via GitHub
albertobastos commented on code in PR #15188: URL: https://github.com/apache/pinot/pull/15188#discussion_r1981074948 ## pinot-controller/src/main/resources/app/components/Query/VisualizeQueryStageStats.tsx: ## @@ -133,8 +133,7 @@ const generateFlowElements = (stats) => { cons

Re: [PR] Refactor query error handling to use QueryErrorCode and QueryErrorMessage for improved clarity and consistency [pinot]

2025-03-05 Thread via GitHub
gortiz commented on PR #15037: URL: https://github.com/apache/pinot/pull/15037#issuecomment-2700595639 > I didn't follow why we reintroduced the truncated stack traces in the error data blocks here? Are you planning to remove it in a follow-up PR instead? Any reason we couldn't have removed

Re: [PR] Refactor query error handling to use QueryErrorCode and QueryErrorMessage for improved clarity and consistency [pinot]

2025-03-05 Thread via GitHub
gortiz commented on code in PR #15037: URL: https://github.com/apache/pinot/pull/15037#discussion_r1981177108 ## pinot-common/src/main/java/org/apache/pinot/common/response/BrokerResponse.java: ## @@ -76,11 +76,12 @@ default void toOutputStream(OutputStream outputStream) /**

Re: [PR] Do not convert IN/NOT_IN to OR lists [pinot]

2025-03-05 Thread via GitHub
yashmayya commented on code in PR #15027: URL: https://github.com/apache/pinot/pull/15027#discussion_r1981097772 ## pinot-query-planner/src/main/java/org/apache/pinot/calcite/sql/fun/PinotOperatorTable.java: ## @@ -97,6 +97,13 @@ public static PinotOperatorTable instance() {

[I] Pinot Helm Chart 0.3.1 Release [pinot]

2025-03-05 Thread via GitHub
PovilasV1 opened a new issue, #15194: URL: https://github.com/apache/pinot/issues/15194 New changes have been made to the Pinot helm chart. 1. jute.maxBuffer in java opts were added as an example https://github.com/apache/pinot/pull/15047 2. `namespace.yaml` was removed from the templa

[PR] [HELM]: Publish Helm 0.3.1 [pinot]

2025-03-05 Thread via GitHub
PovilasV1 opened a new pull request, #15195: URL: https://github.com/apache/pinot/pull/15195 Reference issue - https://github.com/apache/pinot/issues/15194 - pushes the helm chart to 0.3.1 version - prepares the chart for release 0.3.2 -- This is an automated message from the

Re: [PR] Do not convert IN/NOT_IN to OR lists [pinot]

2025-03-05 Thread via GitHub
yashmayya commented on PR #15027: URL: https://github.com/apache/pinot/pull/15027#issuecomment-2700290398 Ah, actually the crazy plan is not due to the new rules here but in spite of them. This is the same plan created without the rules but with `InSubQueryThreshold` set to `0`. -- This

[I] Data transfer from one table to another real time table [pinot]

2025-03-05 Thread via GitHub
Aashiqps opened a new issue, #15193: URL: https://github.com/apache/pinot/issues/15193 can anybody help with data transfer of one table to another real time table, both the tables have similar schema and consume data from different kafka topics. one table is having 53 gb data with 2066 sege