You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ab...@apache.org on 2025/02/01 23:46:12 UTC

(pinot) branch release-1.3.0-rc updated (bae3fcc899 -> 6f78151d04)

This is an automated email from the ASF dual-hosted git repository.

abhioncbr pushed a change to branch release-1.3.0-rc
in repository https://gitbox.apache.org/repos/asf/pinot.git


    from bae3fcc899 feat(ui): add option to show visualized query stats (#14848)
     new e8e10f9462 Fix NPE caused by MultiStageQueryThrottler (#14805)
     new 6f78151d04 Revert "Bump org.codehaus.mojo:buildnumber-maven-plugin from 3.2.0 to 3.2.1 (…" (#14969)

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java   | 8 ++++++--
 pom.xml                                                           | 2 +-
 2 files changed, 7 insertions(+), 3 deletions(-)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


(pinot) 01/02: Fix NPE caused by MultiStageQueryThrottler (#14805)

Posted by ab...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

abhioncbr pushed a commit to branch release-1.3.0-rc
in repository https://gitbox.apache.org/repos/asf/pinot.git

commit e8e10f9462c32b807b21d9c76b911b4db2e88d45
Author: Jialiang Li <jl...@linkedin.com>
AuthorDate: Mon Jan 13 18:00:29 2025 -0800

    Fix NPE caused by MultiStageQueryThrottler (#14805)
---
 .../org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java   | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java b/pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java
index 2aab062730..fa40afb2db 100644
--- a/pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java
+++ b/pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java
@@ -406,7 +406,9 @@ public abstract class BaseBrokerStarter implements ServiceStartable {
       clusterConfigChangeHandler.init(_spectatorHelixManager);
     }
     _clusterConfigChangeHandlers.add(_queryQuotaManager);
-    _clusterConfigChangeHandlers.add(_multiStageQueryThrottler);
+    if (_multiStageQueryThrottler != null) {
+      _clusterConfigChangeHandlers.add(_multiStageQueryThrottler);
+    }
     for (ClusterChangeHandler idealStateChangeHandler : _idealStateChangeHandlers) {
       idealStateChangeHandler.init(_spectatorHelixManager);
     }
@@ -416,7 +418,9 @@ public abstract class BaseBrokerStarter implements ServiceStartable {
     }
     _externalViewChangeHandlers.add(_routingManager);
     _externalViewChangeHandlers.add(_queryQuotaManager);
-    _externalViewChangeHandlers.add(_multiStageQueryThrottler);
+    if (_multiStageQueryThrottler != null) {
+      _externalViewChangeHandlers.add(_multiStageQueryThrottler);
+    }
     for (ClusterChangeHandler instanceConfigChangeHandler : _instanceConfigChangeHandlers) {
       instanceConfigChangeHandler.init(_spectatorHelixManager);
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


(pinot) 02/02: Revert "Bump org.codehaus.mojo:buildnumber-maven-plugin from 3.2.0 to 3.2.1 (…" (#14969)

Posted by ab...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

abhioncbr pushed a commit to branch release-1.3.0-rc
in repository https://gitbox.apache.org/repos/asf/pinot.git

commit 6f78151d04fd5947de816b4ac1a8dca92ecc2f7d
Author: Abhishek Sharma <ab...@spothero.com>
AuthorDate: Sat Feb 1 15:26:44 2025 -0500

    Revert "Bump org.codehaus.mojo:buildnumber-maven-plugin from 3.2.0 to 3.2.1 (…" (#14969)
    
    This reverts commit d9a0c7786d59406e912abb8b101990e5e07c8bed.
---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index e489bcafc7..6816022a5e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -2255,7 +2255,7 @@
         <plugin>
           <groupId>org.codehaus.mojo</groupId>
           <artifactId>buildnumber-maven-plugin</artifactId>
-          <version>3.2.1</version>
+          <version>3.2.0</version>
         </plugin>
         <plugin>
           <groupId>org.codehaus.mojo</groupId>


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org