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

2025-03-17 Thread via GitHub
yashmayya commented on code in PR #15263: URL: https://github.com/apache/pinot/pull/15263#discussion_r2000253759 ## pinot-query-planner/src/main/java/org/apache/pinot/query/type/TypeSystem.java: ## @@ -62,16 +62,36 @@ public boolean shouldConvertRaggedUnionTypesToVarying() {

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

2025-03-17 Thread via GitHub
yashmayya commented on code in PR #15263: URL: https://github.com/apache/pinot/pull/15263#discussion_r2000253759 ## pinot-query-planner/src/main/java/org/apache/pinot/query/type/TypeSystem.java: ## @@ -62,16 +62,36 @@ public boolean shouldConvertRaggedUnionTypesToVarying() {

Re: [PR] Enhance index and field config validation to block adding bloomfilter on boolean column [pinot]

2025-03-17 Thread via GitHub
yashmayya commented on code in PR #15283: URL: https://github.com/apache/pinot/pull/15283#discussion_r2000213383 ## pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java: ## @@ -1307,6 +1311,40 @@ public void testValidateFieldConfig() {

Re: [PR] Allows segments deletion in build for pauseless tables [pinot]

2025-03-17 Thread via GitHub
KKcorps commented on code in PR #15299: URL: https://github.com/apache/pinot/pull/15299#discussion_r2000214167 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -1175,6 +1179,136 @@ public SuccessResponse updateTi

[I] Segment Not Getting Uploaded to S3 (Deep Storage) [pinot]

2025-03-17 Thread via GitHub
DixitThinkbiz opened a new issue, #15295: URL: https://github.com/apache/pinot/issues/15295 ### Description I am working on a proof-of-concept (POC) where I need to ingest data into Pinot and use S3 as deep storage. Although data ingestion to Pinot via S3 is successful, the segments are

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

2025-03-17 Thread via GitHub
himanish-star commented on PR #15107: URL: https://github.com/apache/pinot/pull/15107#issuecomment-2731552861 Hi @deemoliu @tibrewalpratik17 @klsince can any one of you merge this PR please? -- This is an automated message from the Apache Git Service. To respond to the message, pleas

Re: [PR] Allows segments deletion in buld for pauseless tables [pinot]

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

Re: [PR] Add Murmur2 as an alias for Murmur Partition function. [pinot]

2025-03-17 Thread via GitHub
mayankshriv commented on PR #15298: URL: https://github.com/apache/pinot/pull/15298#issuecomment-2731427154 > Suggest to also have an alias of KafkaMurmur2. > > As this is using the same seed of Kafka murmur function One issue with that is if Kafka changes their implementation,

Re: [PR] Add Murmur2 as an alias for Murmur Partition function. [pinot]

2025-03-17 Thread via GitHub
mayankshriv merged PR #15298: URL: https://github.com/apache/pinot/pull/15298 -- 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] idea for AZ aware [pinot]

2025-03-17 Thread via GitHub
wirybeaver commented on code in PR #15265: URL: https://github.com/apache/pinot/pull/15265#discussion_r1999668899 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/strategy/custom/ZoneAwareSegmentAssignmentStrategy.java: ## @@ -0,0 +1,20

Re: [PR] Add Disk Utilization Info into Rebalance API [pinot]

2025-03-17 Thread via GitHub
J-HowHuang commented on code in PR #15175: URL: https://github.com/apache/pinot/pull/15175#discussion_r1999798954 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java: ## @@ -184,4 +209,85 @@ private RebalancePreCheck

Re: [PR] Add Disk Utilization Info into Rebalance API [pinot]

2025-03-17 Thread via GitHub
J-HowHuang commented on code in PR #15175: URL: https://github.com/apache/pinot/pull/15175#discussion_r1999799459 ## pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java: ## @@ -4199,7 +4209,7 @@ public void testRebalanceDry

Re: [PR] Add Tenants Info to Rebalance API summary [pinot]

2025-03-17 Thread via GitHub
J-HowHuang commented on code in PR #15284: URL: https://github.com/apache/pinot/pull/15284#discussion_r1999788429 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/RebalanceSummaryResult.java: ## @@ -161,6 +170,62 @@ public int getExpectedValueAf

Re: [PR] Consume in order 2 [pinot]

2025-03-17 Thread via GitHub
noob-se7en commented on code in PR #15261: URL: https://github.com/apache/pinot/pull/15261#discussion_r1993340221 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/SemaphoreCoordinator.java: ## @@ -0,0 +1,73 @@ +/** + * Licensed to the Apache Software Found

Re: [PR] Add Tenants Info to Rebalance API summary [pinot]

2025-03-17 Thread via GitHub
J-HowHuang commented on code in PR #15284: URL: https://github.com/apache/pinot/pull/15284#discussion_r1999788429 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/RebalanceSummaryResult.java: ## @@ -161,6 +170,62 @@ public int getExpectedValueAf

Re: [PR] Enhance index and field config validation to block adding bloomfilter on boolean column [pinot]

2025-03-17 Thread via GitHub
ayesheepatra07 commented on code in PR #15283: URL: https://github.com/apache/pinot/pull/15283#discussion_r1999727336 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/TableConfigUtils.java: ## @@ -1199,6 +1216,8 @@ private static void validateFieldConfig

Re: [PR] Retry on segment cannot build [pinot]

2025-03-17 Thread via GitHub
itschrispeck commented on code in PR #15234: URL: https://github.com/apache/pinot/pull/15234#discussion_r1999349551 ## pinot-core/src/test/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManagerTest.java: ## @@ -1022,7 +1027,13 @@ protected void postStopConsu

Re: [PR] Add Tenants Info to Rebalance API summary [pinot]

2025-03-17 Thread via GitHub
somandal commented on code in PR #15284: URL: https://github.com/apache/pinot/pull/15284#discussion_r1999748429 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/RebalanceSummaryResult.java: ## @@ -161,6 +170,62 @@ public int getExpectedValueAfte

Re: [PR] Add Disk Utilization Info into Rebalance API [pinot]

2025-03-17 Thread via GitHub
somandal commented on PR #15175: URL: https://github.com/apache/pinot/pull/15175#issuecomment-2731050670 can you also post an example of what all pre-checks will look like with your new changes? or update your description? -- This is an automated message from the Apache Git Service. To r

Re: [PR] idea for AZ aware [pinot]

2025-03-17 Thread via GitHub
zeronerdzerogeekzerocool commented on code in PR #15265: URL: https://github.com/apache/pinot/pull/15265#discussion_r1999748951 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/strategy/custom/ZoneAwareSegmentAssignmentStrategy.java: ##

Re: [PR] Add Tenants Info to Rebalance API summary [pinot]

2025-03-17 Thread via GitHub
somandal commented on code in PR #15284: URL: https://github.com/apache/pinot/pull/15284#discussion_r1999747297 ## pinot-controller/src/test/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancerClusterStatelessTest.java: ## @@ -27,6 +27,7 @@ import java.util.con

Re: [PR] Add Disk Utilization Info into Rebalance API [pinot]

2025-03-17 Thread via GitHub
somandal commented on code in PR #15175: URL: https://github.com/apache/pinot/pull/15175#discussion_r1999729340 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java: ## @@ -42,19 +47,27 @@ public class DefaultRebalanc

Re: [PR] Retry on segment cannot build [pinot]

2025-03-17 Thread via GitHub
lnbest0707-uber commented on code in PR #15234: URL: https://github.com/apache/pinot/pull/15234#discussion_r1999700724 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java: ## @@ -1093,6 +1093,35 @@ public void seg

[I] [spark connector] [pinot]

2025-03-17 Thread via GitHub
tsekityam opened a new issue, #15300: URL: https://github.com/apache/pinot/issues/15300 Hi team, When I try to use [pinot-spark-3-connector](https://github.com/apache/pinot/tree/master/pinot-connectors/pinot-spark-3-connector) to fetch our pinot table to databricks with the following

Re: [PR] Enhance index and field config validation to block adding bloomfilter on boolean column [pinot]

2025-03-17 Thread via GitHub
ayesheepatra07 commented on code in PR #15283: URL: https://github.com/apache/pinot/pull/15283#discussion_r1999729756 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/TableConfigUtils.java: ## @@ -1243,6 +1262,12 @@ private static void validateFieldConfi

Re: [PR] idea for AZ aware [pinot]

2025-03-17 Thread via GitHub
wirybeaver commented on code in PR #15265: URL: https://github.com/apache/pinot/pull/15265#discussion_r1999668899 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/strategy/custom/ZoneAwareSegmentAssignmentStrategy.java: ## @@ -0,0 +1,20

Re: [PR] Retry on segment cannot build [pinot]

2025-03-17 Thread via GitHub
lnbest0707-uber commented on code in PR #15234: URL: https://github.com/apache/pinot/pull/15234#discussion_r1999706050 ## pinot-core/src/test/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManagerTest.java: ## @@ -1022,7 +1027,13 @@ protected void postStopCo

Re: [PR] idea for AZ aware [pinot]

2025-03-17 Thread via GitHub
wirybeaver commented on code in PR #15265: URL: https://github.com/apache/pinot/pull/15265#discussion_r1999668899 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/strategy/custom/ZoneAwareSegmentAssignmentStrategy.java: ## @@ -0,0 +1,20

Re: [PR] idea for AZ aware [pinot]

2025-03-17 Thread via GitHub
wirybeaver commented on code in PR #15265: URL: https://github.com/apache/pinot/pull/15265#discussion_r1999675496 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/strategy/custom/ZoneAwareSegmentAssignmentStrategy.java: ## @@ -0,0 +1,20

Re: [PR] idea for AZ aware [pinot]

2025-03-17 Thread via GitHub
wirybeaver commented on code in PR #15265: URL: https://github.com/apache/pinot/pull/15265#discussion_r1999675496 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/strategy/custom/ZoneAwareSegmentAssignmentStrategy.java: ## @@ -0,0 +1,20

Re: [PR] idea for AZ aware [pinot]

2025-03-17 Thread via GitHub
wirybeaver commented on code in PR #15265: URL: https://github.com/apache/pinot/pull/15265#discussion_r1999668899 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/strategy/custom/ZoneAwareSegmentAssignmentStrategy.java: ## @@ -0,0 +1,20

Re: [PR] Enhance index and field config validation to block adding bloomfilter on boolean column [pinot]

2025-03-17 Thread via GitHub
klsince commented on code in PR #15283: URL: https://github.com/apache/pinot/pull/15283#discussion_r1999550448 ## pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java: ## @@ -2587,4 +2625,4 @@ public void testNoRGRealtimeTable() {

Re: [PR] Add Disk Utilization Info into Rebalance API [pinot]

2025-03-17 Thread via GitHub
J-HowHuang commented on code in PR #15175: URL: https://github.com/apache/pinot/pull/15175#discussion_r1999452388 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java: ## @@ -138,4 +157,80 @@ private boolean checkIsMi

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

2025-03-17 Thread via GitHub
klsince commented on code in PR #15142: URL: https://github.com/apache/pinot/pull/15142#discussion_r1999439274 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/retention/RetentionManager.java: ## @@ -189,6 +219,110 @@ private boolean shouldDeleteInProgres

Re: [PR] Add Murmur2 as an alias for Murmur Partition function. [pinot]

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

Re: [PR] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
noob-se7en commented on code in PR #15296: URL: https://github.com/apache/pinot/pull/15296#discussion_r1999388578 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -1723,6 +1725,15 @@ public RealtimeSegmentDataManager(

(pinot) branch master updated (9fe11f7bb6 -> 89006f4e86)

2025-03-17 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 9fe11f7bb6 Bump circe.version from 0.14.10 to 0.14.12 (#15289) add 89006f4e86 Bump com.github.luben:zstd-jni fr

Re: [PR] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
noob-se7en commented on code in PR #15296: URL: https://github.com/apache/pinot/pull/15296#discussion_r1999388578 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -1723,6 +1725,15 @@ public RealtimeSegmentDataManager(

[PR] Allows segments deletion in buld for pauseless tables [pinot]

2025-03-17 Thread via GitHub
9aman opened a new pull request, #15299: URL: https://github.com/apache/pinot/pull/15299 (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-mail

Re: [PR] Retry on segment cannot build [pinot]

2025-03-17 Thread via GitHub
itschrispeck commented on code in PR #15234: URL: https://github.com/apache/pinot/pull/15234#discussion_r1999346853 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java: ## @@ -1093,6 +1093,35 @@ public void segmen

Re: [PR] Retry on segment cannot build [pinot]

2025-03-17 Thread via GitHub
itschrispeck commented on code in PR #15234: URL: https://github.com/apache/pinot/pull/15234#discussion_r1999345423 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -868,6 +869,13 @@ public void run() {

Re: [PR] Model Hybrid Table behaviour in SSE query planning using HybridTable class [pinot]

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

[PR] Add Murmur2 as an alias for Murmur Partition function. [pinot]

2025-03-17 Thread via GitHub
mayankshriv opened a new pull request, #15298: URL: https://github.com/apache/pinot/pull/15298 Added Mummur2 as an alias for Murmur Partition Function. - The existing Murmur partition funciton is actually Murmur2. - Users end up using Murmur2 in configs, only to get an error, today.

Re: [PR] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
klsince commented on code in PR #15296: URL: https://github.com/apache/pinot/pull/15296#discussion_r1999319210 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -1066,7 +1067,7 @@ protected SegmentBuildDescriptor buil

Re: [PR] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
klsince commented on code in PR #15296: URL: https://github.com/apache/pinot/pull/15296#discussion_r1999305231 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -1723,6 +1725,15 @@ public RealtimeSegmentDataManager(Seg

Re: [PR] Bump software.amazon.awssdk:bom from 2.30.38 to 2.31.1 [pinot]

2025-03-17 Thread via GitHub
tibrewalpratik17 merged PR #15292: URL: https://github.com/apache/pinot/pull/15292 -- 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...@p

Re: [PR] Bump jna.version from 5.16.0 to 5.17.0 [pinot]

2025-03-17 Thread via GitHub
tibrewalpratik17 merged PR #15288: URL: https://github.com/apache/pinot/pull/15288 -- 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...@p

(pinot) branch master updated: Registering pauseless FSM by default and picking this FSM for pauseless enabled tables (#15241)

2025-03-17 Thread kharekartik
This is an automated email from the ASF dual-hosted git repository. kharekartik 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 eb4d0e1b95 Registering pauseless FSM by defa

[PR] Bump jna.version from 5.16.0 to 5.17.0 [pinot]

2025-03-17 Thread via GitHub
dependabot[bot] opened a new pull request, #15288: URL: https://github.com/apache/pinot/pull/15288 Bumps `jna.version` from 5.16.0 to 5.17.0. Updates `net.java.dev.jna:jna` from 5.16.0 to 5.17.0 Changelog Sourced from https://github.com/java-native-access/jna/blob/master/CHANGES.

Re: [PR] Bump calcite.version from 1.37.0 to 1.39.0 [pinot]

2025-03-17 Thread via GitHub
dependabot[bot] commented on PR #15287: URL: https://github.com/apache/pinot/pull/15287#issuecomment-2729548227 OK, I won't notify you again about this release, but will get in touch when a new version is available. You can also ignore all major, minor, or patch releases for a dependency by

[PR] Model Hybrid Table behaviour in SSE query planning using HybridTable class [pinot]

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

Re: [PR] Add Disk Utilization Info into Rebalance API [pinot]

2025-03-17 Thread via GitHub
somandal commented on code in PR #15175: URL: https://github.com/apache/pinot/pull/15175#discussion_r1999251138 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java: ## @@ -138,4 +157,80 @@ private boolean checkIsMini

Re: [PR] Add Disk Utilization Info into Rebalance API [pinot]

2025-03-17 Thread via GitHub
J-HowHuang commented on code in PR #15175: URL: https://github.com/apache/pinot/pull/15175#discussion_r1999256761 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java: ## @@ -42,19 +47,28 @@ public class DefaultRebala

Re: [PR] Add Disk Utilization Info into Rebalance API [pinot]

2025-03-17 Thread via GitHub
J-HowHuang commented on code in PR #15175: URL: https://github.com/apache/pinot/pull/15175#discussion_r1999248503 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java: ## @@ -138,4 +157,80 @@ private boolean checkIsMi

Re: [PR] Query thread context and cid [pinot]

2025-03-17 Thread via GitHub
gortiz commented on code in PR #15180: URL: https://github.com/apache/pinot/pull/15180#discussion_r1998796609 ## pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/QueryThreadContextIntegrationTest.java: ## @@ -0,0 +1,219 @@ +/** + * Licensed to the Apache

Re: [PR] Improved retention management for hybrid tables [pinot]

2025-03-17 Thread via GitHub
swaminathanmanish commented on code in PR #15257: URL: https://github.com/apache/pinot/pull/15257#discussion_r1999172245 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/retention/RetentionManager.java: ## @@ -118,11 +127,20 @@ private void manageRetentio

(pinot) branch dependabot/maven/org.webjars-swagger-ui-5.20.1 deleted (was 4afea8593f)

2025-03-17 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch dependabot/maven/org.webjars-swagger-ui-5.20.1 in repository https://gitbox.apache.org/repos/asf/pinot.git was 4afea8593f Bump org.webjars:swagger-ui from 5.20.0 to 5.20.1 The revisions

Re: [PR] idea for AZ aware [pinot]

2025-03-17 Thread via GitHub
zeronerdzerogeekzerocool commented on code in PR #15265: URL: https://github.com/apache/pinot/pull/15265#discussion_r1999133442 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/strategy/custom/ZoneAwareSegmentAssignmentStrategy.java: ##

(pinot) branch dependabot/maven/nl.jqno.equalsverifier-equalsverifier-3.19.2 deleted (was 3903d58c01)

2025-03-17 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch dependabot/maven/nl.jqno.equalsverifier-equalsverifier-3.19.2 in repository https://gitbox.apache.org/repos/asf/pinot.git was 3903d58c01 Bump nl.jqno.equalsverifier:equalsverifier from 3

Re: [PR] Bump org.webjars:swagger-ui from 5.20.0 to 5.20.1 [pinot]

2025-03-17 Thread via GitHub
yashmayya merged PR #15294: URL: https://github.com/apache/pinot/pull/15294 -- 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 (89006f4e86 -> d9367482f5)

2025-03-17 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 89006f4e86 Bump com.github.luben:zstd-jni from 1.5.7-1 to 1.5.7-2 (#15293) add d9367482f5 Bump nl.jqno.equalsv

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

2025-03-17 Thread via GitHub
yashmayya merged PR #15290: URL: https://github.com/apache/pinot/pull/15290 -- 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] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
deepthi912 commented on code in PR #15296: URL: https://github.com/apache/pinot/pull/15296#discussion_r1999090987 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -1723,6 +1725,15 @@ public RealtimeSegmentDataManager(

Re: [PR] [feature] render new options for rebalance server [pinot]

2025-03-17 Thread via GitHub
somandal commented on code in PR #15256: URL: https://github.com/apache/pinot/pull/15256#discussion_r1999089433 ## pinot-controller/src/main/resources/app/components/Homepage/Operations/RebalanceServer/RebalanceServerOptions.ts: ## @@ -0,0 +1,150 @@ +export type RebalanceServerO

Re: [PR] [feature] render new options for rebalance server [pinot]

2025-03-17 Thread via GitHub
somandal commented on code in PR #15256: URL: https://github.com/apache/pinot/pull/15256#discussion_r1999088062 ## pinot-controller/src/main/resources/app/components/Homepage/Operations/RebalanceServer/RebalanceServerOptions.ts: ## @@ -0,0 +1,145 @@ +export type RebalanceServerO

Re: [PR] [feature] render new options for rebalance server [pinot]

2025-03-17 Thread via GitHub
himanish-star commented on PR #15256: URL: https://github.com/apache/pinot/pull/15256#issuecomment-2729882087 > @himanish-star can you update the PR description with latest screenshot? also, I think we can increase the dialog width a bit. maybe md size? Done! -- This is an automate

(pinot) branch master updated: Bump org.mockito:mockito-core from 5.16.0 to 5.16.1 (#15291)

2025-03-17 Thread pratik
This is an automated email from the ASF dual-hosted git repository. pratik 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 58389dc2a3 Bump org.mockito:mockito-core from 5.1

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

2025-03-17 Thread via GitHub
gortiz commented on code in PR #15263: URL: https://github.com/apache/pinot/pull/15263#discussion_r1998582208 ## pinot-query-planner/src/main/java/org/apache/pinot/query/type/TypeSystem.java: ## @@ -62,16 +62,36 @@ public boolean shouldConvertRaggedUnionTypesToVarying() { r

Re: [PR] Bump com.github.luben:zstd-jni from 1.5.7-1 to 1.5.7-2 [pinot]

2025-03-17 Thread via GitHub
yashmayya merged PR #15293: URL: https://github.com/apache/pinot/pull/15293 -- 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] [feature] render new options for rebalance server [pinot]

2025-03-17 Thread via GitHub
himanish-star commented on PR #15256: URL: https://github.com/apache/pinot/pull/15256#issuecomment-2729813016 > @himanish-star can you update the PR description with latest screenshot? also, I think we can increase the dialog width a bit. maybe md size? Sure 👍🏻 -- This is an autom

Re: [PR] [feature] render new options for rebalance server [pinot]

2025-03-17 Thread via GitHub
jayeshchoudhary commented on PR #15256: URL: https://github.com/apache/pinot/pull/15256#issuecomment-2729810036 @himanish-star can you update the PR description with latest screenshot? also, I think we can increase the dialog width a bit. maybe md size? -- This is an automated message f

(pinot) branch dependabot/maven/com.github.luben-zstd-jni-1.5.7-2 deleted (was 4e9831fb80)

2025-03-17 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch dependabot/maven/com.github.luben-zstd-jni-1.5.7-2 in repository https://gitbox.apache.org/repos/asf/pinot.git was 4e9831fb80 Bump com.github.luben:zstd-jni from 1.5.7-1 to 1.5.7-2 The

Re: [PR] Bump calcite.version from 1.37.0 to 1.39.0 [pinot]

2025-03-17 Thread via GitHub
yashmayya commented on PR #15287: URL: https://github.com/apache/pinot/pull/15287#issuecomment-2729548508 https://github.com/apache/pinot/pull/15263 -- 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

Re: [PR] Bump parquet.version from 1.15.0 to 1.15.1 [pinot]

2025-03-17 Thread via GitHub
tibrewalpratik17 merged PR #15286: URL: https://github.com/apache/pinot/pull/15286 -- 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...@p

Re: [PR] Query thread context and cid [pinot]

2025-03-17 Thread via GitHub
gortiz commented on code in PR #15180: URL: https://github.com/apache/pinot/pull/15180#discussion_r1998812138 ## pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/QueryThreadContextIntegrationTest.java: ## @@ -0,0 +1,219 @@ +/** + * Licensed to the Apache

Re: [PR] Bump org.mockito:mockito-core from 5.16.0 to 5.16.1 [pinot]

2025-03-17 Thread via GitHub
tibrewalpratik17 merged PR #15291: URL: https://github.com/apache/pinot/pull/15291 -- 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...@p

(pinot) branch dependabot/maven/circe.version-0.14.12 deleted (was b2cf6e8450)

2025-03-17 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch dependabot/maven/circe.version-0.14.12 in repository https://gitbox.apache.org/repos/asf/pinot.git was b2cf6e8450 Bump circe.version from 0.14.10 to 0.14.12 The revisions that were on th

Re: [PR] Query thread context and cid [pinot]

2025-03-17 Thread via GitHub
gortiz commented on code in PR #15180: URL: https://github.com/apache/pinot/pull/15180#discussion_r1998764480 ## pinot-query-planner/src/main/java/org/apache/pinot/query/function/InternalMseFunctions.java: ## @@ -0,0 +1,60 @@ +/** + * Licensed to the Apache Software Foundation (

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

2025-03-17 Thread via GitHub
tibrewalpratik17 commented on PR #15290: URL: https://github.com/apache/pinot/pull/15290#issuecomment-2729549856 @dependabot rebase -- 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 co

Re: [PR] Bump circe.version from 0.14.10 to 0.14.12 [pinot]

2025-03-17 Thread via GitHub
yashmayya merged PR #15289: URL: https://github.com/apache/pinot/pull/15289 -- 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: Bump software.amazon.awssdk:bom from 2.30.38 to 2.31.1 (#15292)

2025-03-17 Thread pratik
This is an automated email from the ASF dual-hosted git repository. pratik 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 6051e252fb Bump software.amazon.awssdk:bom from 2

Re: [PR] Bump calcite.version from 1.37.0 to 1.39.0 [pinot]

2025-03-17 Thread via GitHub
dependabot[bot] closed pull request #15287: Bump calcite.version from 1.37.0 to 1.39.0 URL: https://github.com/apache/pinot/pull/15287 -- 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 co

(pinot) branch master updated (6051e252fb -> a20f640ef3)

2025-03-17 Thread pratik
This is an automated email from the ASF dual-hosted git repository. pratik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from 6051e252fb Bump software.amazon.awssdk:bom from 2.30.38 to 2.31.1 (#15292) add a20f640ef3 Bump jna.version from 5

Re: [PR] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
codecov-commenter commented on PR #15296: URL: https://github.com/apache/pinot/pull/15296#issuecomment-2729538602 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15296?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 parquet.version from 1.15.0 to 1.15.1 (#15286)

2025-03-17 Thread pratik
This is an automated email from the ASF dual-hosted git repository. pratik 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 f3478acef5 Bump parquet.version from 1.15.0 to 1.

Re: [PR] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
noob-se7en commented on code in PR #15296: URL: https://github.com/apache/pinot/pull/15296#discussion_r1998662365 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -1723,6 +1725,15 @@ public RealtimeSegmentDataManager(

Re: [PR] Donot allow consumption for Dedup Tables enabled during segment download and replacement [pinot]

2025-03-17 Thread via GitHub
noob-se7en commented on code in PR #15268: URL: https://github.com/apache/pinot/pull/15268#discussion_r1998686009 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -1725,6 +1723,20 @@ public RealtimeSegmentDataManager(

Re: [PR] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
noob-se7en commented on code in PR #15296: URL: https://github.com/apache/pinot/pull/15296#discussion_r1998662365 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java: ## @@ -1723,6 +1725,15 @@ public RealtimeSegmentDataManager(

[PR] Allows consumption during build for Dedup [pinot]

2025-03-17 Thread via GitHub
noob-se7en opened a new pull request, #15296: URL: https://github.com/apache/pinot/pull/15296 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`

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

2025-03-17 Thread via GitHub
yashmayya commented on code in PR #15263: URL: https://github.com/apache/pinot/pull/15263#discussion_r1998605729 ## pinot-query-planner/src/main/java/org/apache/pinot/query/type/TypeSystem.java: ## @@ -62,16 +62,36 @@ public boolean shouldConvertRaggedUnionTypesToVarying() {

Re: [PR] Add test case for lookup join [pinot]

2025-03-17 Thread via GitHub
krishan1390 commented on code in PR #15244: URL: https://github.com/apache/pinot/pull/15244#discussion_r1998592774 ## pinot-integration-tests/src/test/resources/baseballStats_data.csv: ## Review Comment: yes was previously comparing the actual result of the join and seemed

Re: [I] Failed to execute SegmentGenerationAndPushTask unable store segment to S3 [pinot]

2025-03-17 Thread via GitHub
DixitThinkbiz closed issue #15285: Failed to execute SegmentGenerationAndPushTask unable store segment to S3 URL: https://github.com/apache/pinot/issues/15285 -- 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

(pinot) branch dependabot/maven/nl.jqno.equalsverifier-equalsverifier-3.19.2 created (now 43a47ca5b5)

2025-03-17 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.2 in repository https://gitbox.apache.org/repos/asf/pinot.git at 43a47ca5b5 Bump nl.jqno.equalsverifier:equalsverifier from

[PR] Bump calcite.version from 1.37.0 to 1.39.0 [pinot]

2025-03-17 Thread via GitHub
dependabot[bot] opened a new pull request, #15287: URL: https://github.com/apache/pinot/pull/15287 Bumps `calcite.version` from 1.37.0 to 1.39.0. Updates `org.apache.calcite:calcite-core` from 1.37.0 to 1.39.0 Commits https://github.com/apache/calcite/commit/0b328db78cca67b7b

[PR] Bump circe.version from 0.14.10 to 0.14.12 [pinot]

2025-03-17 Thread via GitHub
dependabot[bot] opened a new pull request, #15289: URL: https://github.com/apache/pinot/pull/15289 Bumps `circe.version` from 0.14.10 to 0.14.12. Updates `io.circe:circe-parser_2.12` from 0.14.10 to 0.14.12 Release notes Sourced from https://github.com/circe/circe/releases";>io.c

(pinot) branch dependabot/maven/parquet.version-1.15.1 created (now 1efbbc9c21)

2025-03-17 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/parquet.version-1.15.1 in repository https://gitbox.apache.org/repos/asf/pinot.git at 1efbbc9c21 Bump parquet.version from 1.15.0 to 1.15.1 No new revisions were added

(pinot) branch dependabot/maven/com.github.luben-zstd-jni-1.5.7-2 created (now 4e9831fb80)

2025-03-17 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.github.luben-zstd-jni-1.5.7-2 in repository https://gitbox.apache.org/repos/asf/pinot.git at 4e9831fb80 Bump com.github.luben:zstd-jni from 1.5.7-1 to 1.5.7-2 No

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.31.0 deleted (was bfd134e121)

2025-03-17 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.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was bfd134e121 Bump software.amazon.awssdk:bom from 2.30.38 to 2.31.0 The

(pinot) branch dependabot/maven/org.webjars-swagger-ui-5.20.1 created (now 4afea8593f)

2025-03-17 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/org.webjars-swagger-ui-5.20.1 in repository https://gitbox.apache.org/repos/asf/pinot.git at 4afea8593f Bump org.webjars:swagger-ui from 5.20.0 to 5.20.1 No new revis

[PR] Bump software.amazon.awssdk:bom from 2.30.38 to 2.31.1 [pinot]

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

  1   2   >