Re: [PR] netty 4.2.0.Final [pinot]

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

[PR] netty 4.2.0.Final [pinot]

2025-04-03 Thread via GitHub
sullis opened a new pull request, #15462: URL: https://github.com/apache/pinot/pull/15462 https://netty.io/news/2025/04/03/4-2-0.html -- 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 t

Re: [PR] netty 4.2.0 rc4 [pinot]

2025-04-03 Thread via GitHub
sullis commented on PR #15227: URL: https://github.com/apache/pinot/pull/15227#issuecomment-2777626159 Obsolete. -- 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 unsubscr

Re: [PR] Make MSE leaf stage use cached TableConfig and Schema from TableDataManager [pinot]

2025-04-03 Thread via GitHub
gortiz commented on code in PR #15461: URL: https://github.com/apache/pinot/pull/15461#discussion_r2028146404 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/offline/DimensionTableDataManager.java: ## @@ -206,8 +206,8 @@ private DimensionTable createFastLookupDime

Re: [PR] netty 4.2.0 rc3 [pinot]

2025-04-03 Thread via GitHub
sullis commented on PR #15164: URL: https://github.com/apache/pinot/pull/15164#issuecomment-2777631055 Netty 4.2.0 Final https://github.com/apache/pinot/pull/15462 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] netty 4.2.0 rc4 [pinot]

2025-04-03 Thread via GitHub
sullis commented on PR #15227: URL: https://github.com/apache/pinot/pull/15227#issuecomment-2777630602 Netty 4.2.0 Final https://github.com/apache/pinot/pull/15462 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

Re: [PR] netty 4.2.0 rc4 [pinot]

2025-04-03 Thread via GitHub
sullis closed pull request #15227: netty 4.2.0 rc4 URL: https://github.com/apache/pinot/pull/15227 -- 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: commi

Re: [PR] [multistage] Add Physical Plan Nodes / Trait Assignment / Logical Agg Rule [pinot]

2025-04-03 Thread via GitHub
ankitsultana commented on code in PR #15439: URL: https://github.com/apache/pinot/pull/15439#discussion_r2028086988 ## pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/traits/PinotExecStrategyTrait.java: ## @@ -0,0 +1,82 @@ +/** + * Licensed to the Apache Software

Re: [PR] [multistage] Add Physical Plan Nodes / Trait Assignment / Logical Agg Rule [pinot]

2025-04-03 Thread via GitHub
ankitsultana commented on code in PR #15439: URL: https://github.com/apache/pinot/pull/15439#discussion_r2028086988 ## pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/traits/PinotExecStrategyTrait.java: ## @@ -0,0 +1,82 @@ +/** + * Licensed to the Apache Software

(pinot) branch hotfix-array-agg updated: Fix Null value for group by queries when no aggregate was called (#15458)

2025-04-03 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch hotfix-array-agg in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/hotfix-array-agg by this push: new e7f418442f Fix Null value for g

(pinot) branch fix-array-agg-group-by created (now 4254e0f9f1)

2025-04-03 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch fix-array-agg-group-by in repository https://gitbox.apache.org/repos/asf/pinot.git at 4254e0f9f1 Fix Null value for group by queries when no aggregate was called This branch includes the fo

Re: [PR] Disabled the disk utilization check periodic task in QuickStart [pinot]

2025-04-03 Thread via GitHub
rajagopr closed pull request #15228: Disabled the disk utilization check periodic task in QuickStart URL: https://github.com/apache/pinot/pull/15228 -- 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 t

(pinot) branch dependabot/maven/org.jacoco-jacoco-maven-plugin-0.8.13 deleted (was 62e3ebea3d)

2025-04-03 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/org.jacoco-jacoco-maven-plugin-0.8.13 in repository https://gitbox.apache.org/repos/asf/pinot.git was 62e3ebea3d Bump org.jacoco:jacoco-maven-plugin from 0.8.12 to 0.8.13

(pinot) branch master updated: Feature: New UI to monitor Rebalance Server Status (#15400)

2025-04-03 Thread xbli
This is an automated email from the ASF dual-hosted git repository. xbli 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 23ae24c0c6 Feature: New UI to monitor Rebalance Ser

Re: [PR] fixing NPE when ArrowResponseEncoder reading the vector data as null [pinot]

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

[I] [multistage] Tracker for Physical Planner Backward Compatibility [pinot]

2025-04-03 Thread via GitHub
ankitsultana opened a new issue, #15455: URL: https://github.com/apache/pinot/issues/15455 This ticket will track b/w compatibility related issues wrt the Physical Planner changes being worked upon in the parent issue. - [ ] Support forced "LOCAL" Exchange Strategy - [ ] Support pa

Re: [PR] [multistage] Add Physical Plan Nodes / Trait Assignment / Logical Agg Rule [pinot]

2025-04-03 Thread via GitHub
ankitsultana commented on code in PR #15439: URL: https://github.com/apache/pinot/pull/15439#discussion_r2027989547 ## pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/traits/TraitAssignment.java: ## @@ -0,0 +1,252 @@ +/** + * Licensed to the Apache Software Founda

Re: [PR] Consuming Rebalance Summary [pinot]

2025-04-03 Thread via GitHub
somandal commented on code in PR #15368: URL: https://github.com/apache/pinot/pull/15368#discussion_r2027973962 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -806,6 +848,153 @@ private List getServerTag(String serv

Re: [PR] [multistage] Add Physical Plan Nodes / Trait Assignment / Logical Agg Rule [pinot]

2025-04-03 Thread via GitHub
itschrispeck commented on code in PR #15439: URL: https://github.com/apache/pinot/pull/15439#discussion_r2027937946 ## pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/traits/TraitAssignment.java: ## @@ -0,0 +1,252 @@ +/** + * Licensed to the Apache Software Founda

Re: [PR] Consuming Rebalance Summary [pinot]

2025-04-03 Thread via GitHub
J-HowHuang commented on code in PR #15368: URL: https://github.com/apache/pinot/pull/15368#discussion_r2027951549 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -806,6 +856,172 @@ private List getServerTag(String se

Re: [PR] Consuming Rebalance Summary [pinot]

2025-04-03 Thread via GitHub
somandal commented on code in PR #15368: URL: https://github.com/apache/pinot/pull/15368#discussion_r2027944455 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/RebalanceSummaryResult.java: ## @@ -306,18 +297,121 @@ public Map getServerSegmentC

Re: [PR] Consuming Rebalance Summary [pinot]

2025-04-03 Thread via GitHub
somandal commented on code in PR #15368: URL: https://github.com/apache/pinot/pull/15368#discussion_r2027942314 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -806,6 +856,172 @@ private List getServerTag(String serv

Re: [PR] Fix Null value for group by queries when no aggregate was called [pinot]

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

[I] [multistage] Tracker for Tests for Physical Planner Changes [pinot]

2025-04-03 Thread via GitHub
ankitsultana opened a new issue, #15456: URL: https://github.com/apache/pinot/issues/15456 Tracking ticket for unit-tests or integration tests that we need to add for the Physical Planner changes. The list will continue to grow as we develop this feature in OSS. -- This is an automated m

Re: [PR] Consuming Rebalance Summary [pinot]

2025-04-03 Thread via GitHub
J-HowHuang commented on code in PR #15368: URL: https://github.com/apache/pinot/pull/15368#discussion_r2027315205 ## pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java: ## @@ -299,7 +299,7 @@ public void setUp() _exec

Re: [PR] Enhance rebalance progress stats [pinot]

2025-04-03 Thread via GitHub
somandal commented on code in PR #15266: URL: https://github.com/apache/pinot/pull/15266#discussion_r2027903591 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/ZkBasedTableRebalanceObserver.java: ## @@ -284,4 +319,204 @@ public static TableReb

Re: [PR] Enhance rebalance progress stats [pinot]

2025-04-03 Thread via GitHub
somandal commented on code in PR #15266: URL: https://github.com/apache/pinot/pull/15266#discussion_r2027904077 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -402,9 +402,17 @@ private RebalanceResult doRebalance(Ta

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027857512 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 Review Comment: I did some investigation and found that some of the google cl

(pinot) branch master updated (0d90951353 -> 08bd321163)

2025-04-03 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from 0d90951353 Bump com.puppycrawl.tools:checkstyle from 10.22.0 to 10.23.0 (#15447) add 08bd321163 Fix Null value for

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027857512 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 Review Comment: I did some investigation and found that some of the google cl

(pinot) branch master updated (2eff87211d -> 0d90951353)

2025-04-03 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 2eff87211d [Refactor] Move segment reload logic into TableDataManager (#15441) add 0d90951353 Bump com.puppycrawl

(pinot) branch fix-array-agg-group-by deleted (was 4254e0f9f1)

2025-04-03 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch fix-array-agg-group-by in repository https://gitbox.apache.org/repos/asf/pinot.git was 4254e0f9f1 Fix Null value for group by queries when no aggregate was called The revisions that were on

(pinot) branch dependabot/maven/com.puppycrawl.tools-checkstyle-10.23.0 deleted (was 450cd854f7)

2025-04-03 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/com.puppycrawl.tools-checkstyle-10.23.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was 450cd854f7 Fix build The revisions that were on this branch are sti

Re: [PR] Bump com.puppycrawl.tools:checkstyle from 10.22.0 to 10.23.0 [pinot]

2025-04-03 Thread via GitHub
Jackie-Jiang merged PR #15447: URL: https://github.com/apache/pinot/pull/15447 -- 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

[I] Automate License and Notice Binary update manual step [pinot]

2025-04-03 Thread via GitHub
abhioncbr opened a new issue, #15460: URL: https://github.com/apache/pinot/issues/15460 Depndabot generally raises the PR to upgrade the dependency if a new version is available. This results in the [License-binary] (https://github.com/apache/pinot/blob/master/LICENSE-binary) and [Notice-b

[I] Automate Pinot release validation process [pinot]

2025-04-03 Thread via GitHub
abhioncbr opened a new issue, #15459: URL: https://github.com/apache/pinot/issues/15459 For the release candidate validation process, we follow 8 or 9 steps, refer [here](https://cwiki.apache.org/confluence/display/PINOT/Validating+a+release+candidate). These steps look very simple and intu

[PR] Fix Null value for group by queries when no aggregate was called [pinot]

2025-04-03 Thread via GitHub
jackjlli opened a new pull request, #15458: URL: https://github.com/apache/pinot/pull/15458 This PR is similar to the previous [PR](https://github.com/apache/pinot/pull/15382) (which fixed the aggregation only queries) but it fixes the one with group-by query pattern. Basically if th

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027798737 ## pom.xml: ## @@ -789,25 +808,37 @@ test-jar + + +org.apache.arrow +arrow-vector +${arrow.version} + +

Re: [PR] Consuming Rebalance Summary [pinot]

2025-04-03 Thread via GitHub
J-HowHuang commented on code in PR #15368: URL: https://github.com/apache/pinot/pull/15368#discussion_r2027794757 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/RebalanceSummaryResult.java: ## @@ -306,18 +297,120 @@ public Map getServerSegmen

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027115818 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 +2.4.21 +2.0.2 +1.3.0 +0.19.0 +8.5.15 +4.0.0 +2.4 +

(pinot) branch dependabot/maven/com.puppycrawl.tools-checkstyle-10.23.0 updated (ae14b33d93 -> 450cd854f7)

2025-04-03 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/com.puppycrawl.tools-checkstyle-10.23.0 in repository https://gitbox.apache.org/repos/asf/pinot.git from ae14b33d93 Bump com.puppycrawl.tools:checkstyle from 10.22.0 to 10.2

(pinot) branch master updated (fcfeec0791 -> 2eff87211d)

2025-04-03 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 fcfeec0791 fixing NPE when ArrowResponseEncoder reading the vector data as null (#15457) add 2eff87211d [Refactor

Re: [PR] [Refactor] Move segment reload logic into TableDataManager [pinot]

2025-04-03 Thread via GitHub
Jackie-Jiang merged PR #15441: URL: https://github.com/apache/pinot/pull/15441 -- 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] Bump com.puppycrawl.tools:checkstyle from 10.22.0 to 10.23.0 [pinot]

2025-04-03 Thread via GitHub
Jackie-Jiang commented on PR #15447: URL: https://github.com/apache/pinot/pull/15447#issuecomment-2776629972 @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 commen

Re: [PR] fixing NPE when ArrowResponseEncoder reading the vector data as null [pinot]

2025-04-03 Thread via GitHub
xiangfu0 merged PR #15457: URL: https://github.com/apache/pinot/pull/15457 -- 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 (ff8e03cf9e -> fcfeec0791)

2025-04-03 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 ff8e03cf9e Bump org.webjars:swagger-ui from 5.20.1 to 5.20.3 (#15451) add fcfeec0791 fixing NPE when ArrowRespons

(pinot) branch master updated (874ceb86f3 -> ff8e03cf9e)

2025-04-03 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 874ceb86f3 Bump com.nimbusds:nimbus-jose-jwt from 10.0.2 to 10.1 (#15450) add ff8e03cf9e Bump org.webjars:swagger

Re: [PR] Direct mse query server [pinot]

2025-04-03 Thread via GitHub
Jackie-Jiang commented on code in PR #15445: URL: https://github.com/apache/pinot/pull/15445#discussion_r2027511719 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/service/server/QueryServer.java: ## @@ -159,37 +164,49 @@ public void submit(Worker.QueryRequest reques

[PR] fixing NPE when ArrowResponseEncoder reading the vector data as null [pinot]

2025-04-03 Thread via GitHub
xiangfu0 opened a new pull request, #15457: URL: https://github.com/apache/pinot/pull/15457 Fix the NPE: https://github.com/apache/pinot/actions/runs/14242077339/job/39914086390?pr=15448 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

(pinot) branch dependabot/maven/org.webjars-swagger-ui-5.20.3 deleted (was fac8b020be)

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

(pinot) branch master updated (2196306385 -> d6c1713045)

2025-04-03 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 2196306385 Bump software.amazon.awssdk:bom from 2.31.11 to 2.31.14 (#15448) add d6c1713045 Bump org.jacoco:jacoco

Re: [PR] Extend POSSIBLE_QUERY_OPTIONS in PinotConnection to all query level o… [pinot]

2025-04-03 Thread via GitHub
chrajeshbabu commented on PR #15281: URL: https://github.com/apache/pinot/pull/15281#issuecomment-2776271245 Thanks you @Jackie-Jiang fixed the minor nits. Now it's good to go for merging. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Enhance rebalance progress stats [pinot]

2025-04-03 Thread via GitHub
klsince commented on code in PR #15266: URL: https://github.com/apache/pinot/pull/15266#discussion_r2027561893 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalanceProgressStats.java: ## @@ -118,6 +128,116 @@ public RebalanceStateStats

Re: [PR] Bump com.nimbusds:nimbus-jose-jwt from 10.0.2 to 10.1 [pinot]

2025-04-03 Thread via GitHub
Jackie-Jiang merged PR #15450: URL: https://github.com/apache/pinot/pull/15450 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot

(pinot) branch dependabot/maven/com.nimbusds-nimbus-jose-jwt-10.1 deleted (was 8a0c1f1bdc)

2025-04-03 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/com.nimbusds-nimbus-jose-jwt-10.1 in repository https://gitbox.apache.org/repos/asf/pinot.git was 8a0c1f1bdc Bump com.nimbusds:nimbus-jose-jwt from 10.0.2 to 10.1 The revi

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.31.14 deleted (was 670de9ff33)

2025-04-03 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/software.amazon.awssdk-bom-2.31.14 in repository https://gitbox.apache.org/repos/asf/pinot.git was 670de9ff33 Bump software.amazon.awssdk:bom from 2.31.11 to 2.31.14 The r

Re: [PR] Consuming Rebalance Summary [pinot]

2025-04-03 Thread via GitHub
klsince commented on code in PR #15368: URL: https://github.com/apache/pinot/pull/15368#discussion_r2027503512 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java: ## @@ -806,6 +848,153 @@ private List getServerTag(String serve

[PR] Bump org.jacoco:jacoco-maven-plugin from 0.8.12 to 0.8.13 [pinot]

2025-04-03 Thread via GitHub
dependabot[bot] opened a new pull request, #15449: URL: https://github.com/apache/pinot/pull/15449 Bumps [org.jacoco:jacoco-maven-plugin](https://github.com/jacoco/jacoco) from 0.8.12 to 0.8.13. Release notes Sourced from https://github.com/jacoco/jacoco/releases";>org.jacoco:jacoc

Re: [PR] Enhance rebalance progress stats [pinot]

2025-04-03 Thread via GitHub
J-HowHuang commented on code in PR #15266: URL: https://github.com/apache/pinot/pull/15266#discussion_r2027412227 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalanceProgressStats.java: ## @@ -19,33 +19,25 @@ package org.apache.pinot

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027407183 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 Review Comment: It would be good to pin the version of com.google.api.grpc:p

Re: [PR] Allow lazy init of the record reader in CompactedPinotSegmentRecordReader [pinot]

2025-04-03 Thread via GitHub
swaminathanmanish merged PR #15428: URL: https://github.com/apache/pinot/pull/15428 -- 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) branch master updated (4f91c27120 -> 404e50e887)

2025-04-03 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 4f91c27120 Adding ArrowResponseEncoder implementation (#15410) add 404e50e887 Allow lazy init of the re

Re: [I] Fix critical CVEs in 1.3.0 [pinot]

2025-04-03 Thread via GitHub
piby180 commented on issue #15453: URL: https://github.com/apache/pinot/issues/15453#issuecomment-2775785875 Here is a list of all CVEs [Vulnerability_Report Apr 02 2025 17_12.pdf](https://github.com/user-attachments/files/19587990/Vulnerability_Report.Apr.02.2025.17_12.pdf) -- Thi

(pinot) branch dependabot/maven/org.webjars-swagger-ui-5.20.3 created (now fac8b020be)

2025-04-03 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.3 in repository https://gitbox.apache.org/repos/asf/pinot.git at fac8b020be Bump org.webjars:swagger-ui from 5.20.1 to 5.20.3 No new revis

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027063857 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 +2.4.21 +2.0.2 +1.3.0 Review Comment: Try to use the same naming

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027119008 ## pom.xml: ## @@ -792,22 +810,26 @@ org.checkerframework checker-qual -3.49.2 +${checker-qual.version} + Review Comment:

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027115818 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 +2.4.21 +2.0.2 +1.3.0 +0.19.0 +8.5.15 +4.0.0 +2.4 +

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027112915 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 +2.4.21 +2.0.2 +1.3.0 +0.19.0 +8.5.15 +4.0.0 +2.4 +

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027080536 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 +2.4.21 +2.0.2 +1.3.0 +0.19.0 Review Comment: Suggest replaci

Re: [PR] Moving dependency version declarations to maven properties [pinot]

2025-04-03 Thread via GitHub
gviedma commented on code in PR #15437: URL: https://github.com/apache/pinot/pull/15437#discussion_r2027063857 ## pom.xml: ## @@ -271,6 +271,24 @@ 2.5.3 0.11.0-pre2 0.3.2 +3.49.2 +2.4.21 +2.0.2 +1.3.0 Review Comment: Try to use the same naming

Re: [PR] (Minor) fix logs [pinot]

2025-04-03 Thread via GitHub
noob-se7en commented on code in PR #15438: URL: https://github.com/apache/pinot/pull/15438#discussion_r2026891538 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java: ## @@ -621,7 +621,7 @@ private void commitSegm

Re: [PR] (Minor) fix logs [pinot]

2025-04-03 Thread via GitHub
noob-se7en commented on code in PR #15438: URL: https://github.com/apache/pinot/pull/15438#discussion_r2026873578 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java: ## @@ -621,7 +621,7 @@ private void commitSegm

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.31.13 deleted (was f2c0b22856)

2025-04-03 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.13 in repository https://gitbox.apache.org/repos/asf/pinot.git was f2c0b22856 Bump software.amazon.awssdk:bom from 2.31.11 to 2.31.13 T

[PR] Bump software.amazon.awssdk:bom from 2.31.11 to 2.31.14 [pinot]

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

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

2025-04-03 Thread via GitHub
dependabot[bot] opened a new pull request, #15451: URL: https://github.com/apache/pinot/pull/15451 Bumps [org.webjars:swagger-ui](https://github.com/swagger-api/swagger-ui) from 5.20.1 to 5.20.3. Release notes Sourced from https://github.com/swagger-api/swagger-ui/releases";>org.we

Re: [PR] Bump software.amazon.awssdk:bom from 2.31.11 to 2.31.13 [pinot]

2025-04-03 Thread via GitHub
dependabot[bot] closed pull request #15435: Bump software.amazon.awssdk:bom from 2.31.11 to 2.31.13 URL: https://github.com/apache/pinot/pull/15435 -- 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

[PR] Bump com.nimbusds:nimbus-jose-jwt from 10.0.2 to 10.1 [pinot]

2025-04-03 Thread via GitHub
dependabot[bot] opened a new pull request, #15450: URL: https://github.com/apache/pinot/pull/15450 Bumps [com.nimbusds:nimbus-jose-jwt](https://bitbucket.org/connect2id/nimbus-jose-jwt) from 10.0.2 to 10.1. Changelog Sourced from https://bitbucket.org/connect2id/nimbus-jose-jwt/sr

(pinot) branch dependabot/maven/com.nimbusds-nimbus-jose-jwt-10.1 created (now 8a0c1f1bdc)

2025-04-03 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.nimbusds-nimbus-jose-jwt-10.1 in repository https://gitbox.apache.org/repos/asf/pinot.git at 8a0c1f1bdc Bump com.nimbusds:nimbus-jose-jwt from 10.0.2 to 10.1 No n

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.31.14 created (now 670de9ff33)

2025-04-03 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.14 in repository https://gitbox.apache.org/repos/asf/pinot.git at 670de9ff33 Bump software.amazon.awssdk:bom from 2.31.11 to 2.31.14 N

(pinot) branch dependabot/maven/org.jacoco-jacoco-maven-plugin-0.8.13 created (now 62e3ebea3d)

2025-04-03 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.jacoco-jacoco-maven-plugin-0.8.13 in repository https://gitbox.apache.org/repos/asf/pinot.git at 62e3ebea3d Bump org.jacoco:jacoco-maven-plugin from 0.8.12 to 0.8.

Re: [PR] Bump software.amazon.awssdk:bom from 2.31.11 to 2.31.13 [pinot]

2025-04-03 Thread via GitHub
dependabot[bot] commented on PR #15435: URL: https://github.com/apache/pinot/pull/15435#issuecomment-2775484566 Superseded by #15448. -- 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

(pinot) branch dependabot/maven/com.puppycrawl.tools-checkstyle-10.23.0 created (now 526e0af93b)

2025-04-03 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.puppycrawl.tools-checkstyle-10.23.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at 526e0af93b Bump com.puppycrawl.tools:checkstyle from 10.22.0 to

[PR] Bump com.puppycrawl.tools:checkstyle from 10.22.0 to 10.23.0 [pinot]

2025-04-03 Thread via GitHub
dependabot[bot] opened a new pull request, #15447: URL: https://github.com/apache/pinot/pull/15447 Bumps [com.puppycrawl.tools:checkstyle](https://github.com/checkstyle/checkstyle) from 10.22.0 to 10.23.0. Release notes Sourced from https://github.com/checkstyle/checkstyle/release

Re: [PR] Remove "is not resolvable to a concrete type." warning [pinot]

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

Re: [I] [discussion] On Upsert Hybrid Tables [pinot]

2025-04-03 Thread via GitHub
rajat-sr1704 commented on issue #12261: URL: https://github.com/apache/pinot/issues/12261#issuecomment-2774895808 In this case if we want to store 3 months of data in Realtime table, we cannot use star tree index on the Realtime table, right? we can make this star tree index work with offli

[PR] Remove "is not resolvable to a concrete type." warning [pinot]

2025-04-03 Thread via GitHub
vrajat opened a new pull request, #15446: URL: https://github.com/apache/pinot/pull/15446 Specifically, the following warnings have been removed by using change the method signatures to return concrete types. ``` WARNING: Return type, java.util.Collection, of method, public java.u

Re: [PR] Direct mse query server [pinot]

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

[PR] Direct mse query server [pinot]

2025-04-03 Thread via GitHub
gortiz opened a new pull request, #15445: URL: https://github.com/apache/pinot/pull/15445 We have detected production cases where the number of GRPC and query submission threads is too high. There are two reasons for that: 1. We are blocking submissions threads when we shouldn't. 2. We