This is an automated email from the ASF dual-hosted git repository.
xiangfu 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 1393750622 Allow Server throttling just before executing queries on
server to allow max CPU and disk utilization (#12930)
1393750622 is described below
commit 1393750622462b8a80dfc0550de3d3723f2bba26
Author: Chaitanya Deepthi <[email protected]>
AuthorDate: Tue Apr 16 02:39:15 2024 -0400
Allow Server throttling just before executing queries on server to allow
max CPU and disk utilization (#12930)
Co-authored-by: deepthi912 <[email protected]>
---
.../org/apache/pinot/server/starter/helix/BaseServerStarter.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java
b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java
index f16b326591..02c7b81ea5 100644
---
a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java
+++
b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java
@@ -578,9 +578,6 @@ public abstract class BaseServerStarter implements
ServiceStartable {
_serverInstance = new ServerInstance(serverConf, _helixManager,
accessControlFactory);
ServerMetrics serverMetrics = _serverInstance.getServerMetrics();
- // Enable Server level realtime ingestion rate limier
-
RealtimeConsumptionRateManager.getInstance().createServerRateLimiter(_serverConf,
serverMetrics);
-
InstanceDataManager instanceDataManager =
_serverInstance.getInstanceDataManager();
instanceDataManager.setSupplierOfIsServerReadyToServeQueries(() ->
_isServerReadyToServeQueries);
// initialize the thread accountant for query killing
@@ -639,6 +636,9 @@ public abstract class BaseServerStarter implements
ServiceStartable {
preServeQueries();
+ // Enable Server level realtime ingestion rate limier
+
RealtimeConsumptionRateManager.getInstance().createServerRateLimiter(_serverConf,
serverMetrics);
+
// Start the query server after finishing the service status check. If the
query server is started before all the
// segments are loaded, broker might not have finished processing the
callback of routing table update, and start
// querying the server pre-maturely.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
