Re: [PR] protect usage MSQE compiler for empty schema polyfill with config param [pinot]

2025-02-18 Thread via GitHub
albertobastos commented on code in PR #15078: URL: https://github.com/apache/pinot/pull/15078#discussion_r1961128048 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java: ## @@ -146,6 +146,7 @@ public abstract class BaseSi

(pinot) branch master updated (ce7524c8b0 -> 802226ec8c)

2025-02-18 Thread manishswaminathan
This is an automated email from the ASF dual-hosted git repository. manishswaminathan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from ce7524c8b0 Improve FailureDetector logic in QueryDispatcher for MSQE (#15084) add 802226ec8c Emits met

Re: [PR] Emits metric before Consumer gets stuck in postStopConsumedMsg Method [pinot]

2025-02-18 Thread via GitHub
swaminathanmanish merged PR #15070: URL: https://github.com/apache/pinot/pull/15070 -- 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...@

Re: [I] Facing issue while removing pinot server [pinot]

2025-02-18 Thread via GitHub
rohitraj-carousellgroup commented on issue #14944: URL: https://github.com/apache/pinot/issues/14944#issuecomment-2667487243 can we have any update? -- 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 t

(pinot) branch master updated: Improve FailureDetector logic in QueryDispatcher for MSQE (#15084)

2025-02-18 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 ce7524c8b0 Improve FailureDetector logic in Qu

Re: [PR] Improve FailureDetector logic in QueryDispatcher for MSQE [pinot]

2025-02-18 Thread via GitHub
yashmayya merged PR #15084: URL: https://github.com/apache/pinot/pull/15084 -- 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] Decouple tracking and storage in MinionEventObserver using MinionTaskObserverStorageManager [pinot]

2025-02-18 Thread via GitHub
rajagopr commented on code in PR #15044: URL: https://github.com/apache/pinot/pull/15044#discussion_r1960790378 ## pinot-minion/src/main/java/org/apache/pinot/minion/BaseMinionStarter.java: ## @@ -168,6 +172,26 @@ public void registerEventObserverFactory(MinionEventObserverFact

Re: [PR] protect usage MSQE compiler for empty schema polyfill with config param [pinot]

2025-02-18 Thread via GitHub
Jackie-Jiang commented on PR #15078: URL: https://github.com/apache/pinot/pull/15078#issuecomment-2667270674 Seems we already use `mse` instead of `msqe` in some config (e.g. `pinot.server.mse.max.init.group.holder.capacity`) and query option (e.g. `mseMaxInitialResultHolderCapacity`), cons

Re: [PR] Release 1.3.0 download update. [pinot-site]

2025-02-18 Thread via GitHub
xiangfu0 commented on code in PR #141: URL: https://github.com/apache/pinot-site/pull/141#discussion_r1960786067 ## data/downloads/1.3.0.mdx: ## @@ -0,0 +1,16 @@ +--- +version: 1.3.0 +date: 17/02/2025 Review Comment: The date conversion is different, we should use 2/17/2025

(pinot-site) branch new-site-dev updated: fix the date for download 1.3.0 page

2025-02-18 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 984ba561 fix the date for downl

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
somandal commented on PR #15050: URL: https://github.com/apache/pinot/pull/15050#issuecomment-2667227134 > points 3,4,5 look great. > point 2 - sure we can take this up later > For 1, the tag in each server's section is helpful. What I was suggesting though, is that we provide the list

(pinot-site) branch new-site-dev-1.3.0 deleted (was e9c5695a)

2025-02-18 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch new-site-dev-1.3.0 in repository https://gitbox.apache.org/repos/asf/pinot-site.git was e9c5695a update more pages references to 1.3.0 release The revisions that were on this branch are st

(pinot-site) branch new-site-dev updated: update more pages references to 1.3.0 release (#142)

2025-02-18 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 f9ebe12b update more pages refe

Re: [PR] update more pages references to 1.3.0 release [pinot-site]

2025-02-18 Thread via GitHub
xiangfu0 merged PR #142: URL: https://github.com/apache/pinot-site/pull/142 -- 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-static-prod updated (8fdb6738 -> 395b998d)

2025-02-18 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch new-static-prod in repository https://gitbox.apache.org/repos/asf/pinot-site.git discard 8fdb6738 remove old static folder This update removed existing revisions from the reference, leaving th

(pinot-site) 01/01: update more pages references to 1.3.0 release

2025-02-18 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch new-site-dev-1.3.0 in repository https://gitbox.apache.org/repos/asf/pinot-site.git commit e9c5695aef924460ca2c499d8de60a05cfc23300 Author: Xiang Fu AuthorDate: Tue Feb 18 16:23:36 2025 -0800

(pinot-site) branch new-site-dev-1.3.0 created (now e9c5695a)

2025-02-18 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch new-site-dev-1.3.0 in repository https://gitbox.apache.org/repos/asf/pinot-site.git at e9c5695a update more pages references to 1.3.0 release This branch includes the following new commit

[PR] update more pages references to 1.3.0 release [pinot-site]

2025-02-18 Thread via GitHub
xiangfu0 opened a new pull request, #142: URL: https://github.com/apache/pinot-site/pull/142 (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-

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
somandal commented on code in PR #15050: URL: https://github.com/apache/pinot/pull/15050#discussion_r1960752411 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -559,6 +588,147 @@ private RebalanceResult doRebalance(T

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
somandal commented on code in PR #15050: URL: https://github.com/apache/pinot/pull/15050#discussion_r1960751841 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -559,6 +588,147 @@ private RebalanceResult doRebalance(T

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
somandal commented on code in PR #15050: URL: https://github.com/apache/pinot/pull/15050#discussion_r1960752153 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -559,6 +588,147 @@ private RebalanceResult doRebalance(T

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
somandal commented on code in PR #15050: URL: https://github.com/apache/pinot/pull/15050#discussion_r1960751514 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -559,6 +588,147 @@ private RebalanceResult doRebalance(T

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
somandal commented on code in PR #15050: URL: https://github.com/apache/pinot/pull/15050#discussion_r1960751277 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -291,43 +305,58 @@ private RebalanceResult doRebalance(T

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
npawar commented on PR #15050: URL: https://github.com/apache/pinot/pull/15050#issuecomment-2667186163 > Updated summary: > > Adding a server: > > ``` > { > "serverInfo" : { > "numServersGettingNewSegments" : 1, > "numServers" : { > "valueBeforeReb

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
somandal commented on code in PR #15050: URL: https://github.com/apache/pinot/pull/15050#discussion_r1960750727 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -559,6 +588,147 @@ private RebalanceResult doRebalance(T

Re: [PR] Delete Old Files containing Google Analytics in new-static-prod branch [pinot-site]

2025-02-18 Thread via GitHub
xiangfu0 closed pull request #140: Delete Old Files containing Google Analytics in new-static-prod branch URL: https://github.com/apache/pinot-site/pull/140 -- 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] Delete Old Files containing Google Analytics in new-static-prod branch [pinot-site]

2025-02-18 Thread via GitHub
xiangfu0 commented on PR #140: URL: https://github.com/apache/pinot-site/pull/140#issuecomment-2667180534 updated here: https://github.com/apache/pinot-site/commit/f69db93df046d13cb72b020f50e8000731a4601d -- This is an automated message from the Apache Git Service. To respond to the messa

(pinot-site) branch new-static-prod updated (08a084c7 -> f69db93d)

2025-02-18 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch new-static-prod in repository https://gitbox.apache.org/repos/asf/pinot-site.git from 08a084c7 Update Pinot Site from dev branch 25bd6b5c6922fb93f9cd73e7fe953bfa6c9be302 add f69db93d r

Re: [PR] Delete Old Files containing Google Analytics in new-static-prod branch [pinot-site]

2025-02-18 Thread via GitHub
xiangfu0 commented on PR #140: URL: https://github.com/apache/pinot-site/pull/140#issuecomment-2667172196 Thanks! I will modify the code from this branch: [new-site-dev](https://github.com/apache/pinot-site/tree/new-site-dev) -- This is an automated message from the Apache Git Service

(pinot-site) branch new-static-prod updated (d8887ce8 -> 08a084c7)

2025-02-18 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch new-static-prod in repository https://gitbox.apache.org/repos/asf/pinot-site.git from d8887ce8 Update Pinot Site from dev branch f9f34626d1a247a6fd9d0d16935a2a81ca12f014 add 08a084c

(pinot-site) branch new-site-dev updated: Release 1.3.0 download update. (#141)

2025-02-18 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 25bd6b5c Release 1.3.0 download

Re: [PR] Release 1.3.0 download update. [pinot-site]

2025-02-18 Thread via GitHub
xiangfu0 merged PR #141: URL: https://github.com/apache/pinot-site/pull/141 -- 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] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
klsince commented on code in PR #15050: URL: https://github.com/apache/pinot/pull/15050#discussion_r1960684388 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -291,43 +305,58 @@ private RebalanceResult doRebalance(Ta

(pinot) branch master updated: Fixing aliased columns bug in GAPFILL based queries (#15052)

2025-02-18 Thread ankitsultana
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 dd37e6490d Fixing aliased columns bug in GA

Re: [PR] Fixing aliased columns bug in GAPFILL based queries [pinot]

2025-02-18 Thread via GitHub
ankitsultana merged PR #15052: URL: https://github.com/apache/pinot/pull/15052 -- 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 master updated (946abeb451 -> d76fe22842)

2025-02-18 Thread jackie
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 946abeb451 Adding broker grpc query endpoint and BrokerRequest/BrokerResponse protobuf (#15081) add d76fe22842 Va

Re: [PR] Validate Table Replication During Create/Update [pinot]

2025-02-18 Thread via GitHub
Jackie-Jiang merged PR #13951: URL: https://github.com/apache/pinot/pull/13951 -- 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] [kinesis] Disabled testGetStreamNamesWithPagination on CI due to cost [pinot]

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

Re: [PR] Add required proto for broker grpc query server and request/response objects [pinot]

2025-02-18 Thread via GitHub
xiangfu0 merged PR #15081: URL: https://github.com/apache/pinot/pull/15081 -- 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 (bfd511a4b6 -> 946abeb451)

2025-02-18 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 bfd511a4b6 Compatibility verification against release 1.3.0 (#15080) add 946abeb451 Adding broker grpc query endp

Re: [PR] protect usage MSQE compiler for empty schema polyfill with config param [pinot]

2025-02-18 Thread via GitHub
Jackie-Jiang commented on code in PR #15078: URL: https://github.com/apache/pinot/pull/15078#discussion_r1960454941 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java: ## @@ -146,6 +146,7 @@ public abstract class BaseSin

Re: [PR] protect usage MSQE compiler for empty schema polyfill with config param [pinot]

2025-02-18 Thread via GitHub
Jackie-Jiang commented on code in PR #15078: URL: https://github.com/apache/pinot/pull/15078#discussion_r1958766725 ## pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java: ## @@ -584,6 +584,10 @@ public static class QueryOptionKey { // Custom Query

(pinot) branch master updated (6d5159b023 -> bfd511a4b6)

2025-02-18 Thread jackie
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 6d5159b023 Bump nl.jqno.equalsverifier:equalsverifier from 3.19 to 3.19.1 (#15082) add bfd511a4b6 Compatibility v

Re: [PR] Compatibility verification against release 1.3.0 [pinot]

2025-02-18 Thread via GitHub
Jackie-Jiang merged PR #15080: URL: https://github.com/apache/pinot/pull/15080 -- 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] [kinesis] Disabled testGetStreamNamesWithPagination on CI due to cost [pinot]

2025-02-18 Thread via GitHub
suvodeep-pyne opened a new pull request, #15085: URL: https://github.com/apache/pinot/pull/15085 The createStream() call in this test is incurring recurring costs in CI. The test is retained for manual use. Note that the test only runs when AWS secrets are loaded in the env. -- Thi

(pinot) branch master updated (0b5600db25 -> 6d5159b023)

2025-02-18 Thread jackie
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 0b5600db25 Add null checks when sampling thread resource usage. (#15069) add 6d5159b023 Bump nl.jqno.equalsverifi

(pinot) branch dependabot/maven/nl.jqno.equalsverifier-equalsverifier-3.19.1 deleted (was 7719fa8c5c)

2025-02-18 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/nl.jqno.equalsverifier-equalsverifier-3.19.1 in repository https://gitbox.apache.org/repos/asf/pinot.git was 7719fa8c5c Bump nl.jqno.equalsverifier:equalsverifier from 3.19

Re: [PR] Bump nl.jqno.equalsverifier:equalsverifier from 3.19 to 3.19.1 [pinot]

2025-02-18 Thread via GitHub
Jackie-Jiang merged PR #15082: URL: https://github.com/apache/pinot/pull/15082 -- 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] Validate Table Replication During Create/Update [pinot]

2025-02-18 Thread via GitHub
suddendust commented on code in PR #13951: URL: https://github.com/apache/pinot/pull/13951#discussion_r1960038444 ## pinot-common/src/main/java/org/apache/pinot/common/assignment/InstancePartitionsUtils.java: ## @@ -137,7 +137,7 @@ public static InstancePartitions computeDefaul

Re: [PR] Improve FailureDetector logic in QueryDispatcher for MSQE [pinot]

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

Re: [PR] Add a dry-run summary mode for TableRebalance which only returns a summary of the dry-run results [pinot]

2025-02-18 Thread via GitHub
somandal commented on PR #15050: URL: https://github.com/apache/pinot/pull/15050#issuecomment-2666036635 Updated summary: ``` { "serverInfo" : { "numServersGettingNewSegments" : 1, "numServers" : { "valueBeforeRebalance" : 1, "expectedValueAfterR

Re: [PR] Decouple tracking and storage in MinionEventObserver using MinionTaskObserverStorageManager [pinot]

2025-02-18 Thread via GitHub
shounakmk219 commented on code in PR #15044: URL: https://github.com/apache/pinot/pull/15044#discussion_r1959950775 ## pinot-spi/src/main/java/org/apache/pinot/spi/tasks/MinionTaskBaseObserverStats.java: ## @@ -0,0 +1,168 @@ +/** + * Licensed to the Apache Software Foundation (A

[PR] Improve FailureDetector logic in QueryDispatcher for MSQE [pinot]

2025-02-18 Thread via GitHub
yashmayya opened a new pull request, #15084: URL: https://github.com/apache/pinot/pull/15084 - Improves https://github.com/apache/pinot/pull/15005 by updating `QueryDispatcher` to also call the failure detector when there's an exception from the async response and the gRPC channel is unheal

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.30.21 deleted (was a8f85c8b16)

2025-02-18 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.21 in repository https://gitbox.apache.org/repos/asf/pinot.git was a8f85c8b16 Bump software.amazon.awssdk:bom from 2.30.16 to 2.30.21 T

[PR] Bump software.amazon.awssdk:bom from 2.30.16 to 2.30.22 [pinot]

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

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.30.22 created (now 24cdee91b6)

2025-02-18 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.22 in repository https://gitbox.apache.org/repos/asf/pinot.git at 24cdee91b6 Bump software.amazon.awssdk:bom from 2.30.16 to 2.30.22 N

Re: [PR] Bump software.amazon.awssdk:bom from 2.30.16 to 2.30.21 [pinot]

2025-02-18 Thread via GitHub
dependabot[bot] commented on PR #15076: URL: https://github.com/apache/pinot/pull/15076#issuecomment-2665443968 Superseded by #15083. -- 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

Re: [PR] Bump software.amazon.awssdk:bom from 2.30.16 to 2.30.21 [pinot]

2025-02-18 Thread via GitHub
dependabot[bot] closed pull request #15076: Bump software.amazon.awssdk:bom from 2.30.16 to 2.30.21 URL: https://github.com/apache/pinot/pull/15076 -- 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/nl.jqno.equalsverifier-equalsverifier-3.19.1 created (now 7719fa8c5c)

2025-02-18 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/nl.jqno.equalsverifier-equalsverifier-3.19.1 in repository https://gitbox.apache.org/repos/asf/pinot.git at 7719fa8c5c Bump nl.jqno.equalsverifier:equalsverifier from

[PR] Bump nl.jqno.equalsverifier:equalsverifier from 3.19 to 3.19.1 [pinot]

2025-02-18 Thread via GitHub
dependabot[bot] opened a new pull request, #15082: URL: https://github.com/apache/pinot/pull/15082 Bumps [nl.jqno.equalsverifier:equalsverifier](https://github.com/jqno/equalsverifier) from 3.19 to 3.19.1. Release notes Sourced from https://github.com/jqno/equalsverifier/releases"

Re: [PR] Enhance early termination handling in LeafStageTransferableBlockOperator and OpChainSchedulerService [pinot]

2025-02-18 Thread via GitHub
yashmayya commented on code in PR #15071: URL: https://github.com/apache/pinot/pull/15071#discussion_r1959480962 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/executor/OpChainSchedulerService.java: ## @@ -49,10 +49,9 @@ public void register(OpChain operator

Re: [PR] Validate Table Replication During Create/Update [pinot]

2025-02-18 Thread via GitHub
suddendust commented on code in PR #13951: URL: https://github.com/apache/pinot/pull/13951#discussion_r1959417372 ## pinot-controller/src/test/java/org/apache/pinot/controller/api/PinotTableRestletResourceTest.java: ## @@ -746,6 +752,218 @@ public void testUnrecognizedProperties

Re: [PR] Validate Table Replication During Create/Update [pinot]

2025-02-18 Thread via GitHub
suddendust commented on code in PR #13951: URL: https://github.com/apache/pinot/pull/13951#discussion_r1959416531 ## pinot-common/src/main/java/org/apache/pinot/common/assignment/InstancePartitionsUtils.java: ## @@ -161,6 +161,24 @@ public static InstancePartitions computeDefau