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

lakshsingla pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git


The following commit(s) were added to refs/heads/master by this push:
     new e4fdf1055b4 Update default value of 
`druid.indexer.tasklock.batchAllocationWaitTime` to zero (#16578)
e4fdf1055b4 is described below

commit e4fdf1055b4cdbb5ea9d16fdd4130643caa3d9e9
Author: Kashif Faraz <[email protected]>
AuthorDate: Mon Jun 10 07:37:23 2024 -0700

    Update default value of `druid.indexer.tasklock.batchAllocationWaitTime` to 
zero (#16578)
    
    Update default value of druid.indexer.tasklock.batchAllocationWaitTime to 0.
    Thus, a segment allocation request is processed immediately unless there 
are already some requests queued before this one. While in queue, a segment 
allocation request may get clubbed together with other similar requests into a 
batch to reduce load on the metadata store.
---
 docs/configuration/index.md                                             | 2 +-
 .../java/org/apache/druid/indexing/overlord/config/TaskLockConfig.java  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/configuration/index.md b/docs/configuration/index.md
index f8583b95841..1976657c41e 100644
--- a/docs/configuration/index.md
+++ b/docs/configuration/index.md
@@ -1120,7 +1120,7 @@ These Overlord static configurations can be defined in 
the `overlord/runtime.pro
 |`druid.indexer.storage.recentlyFinishedThreshold`|Duration of time to store 
task results. Default is 24 hours. If you have hundreds of tasks running in a 
day, consider increasing this threshold.|`PT24H`|
 |`druid.indexer.tasklock.forceTimeChunkLock`|_**Setting this to false is still 
experimental**_<br/> If set, all tasks are enforced to use time chunk lock. If 
not set, each task automatically chooses a lock type to use. This configuration 
can be overwritten by setting `forceTimeChunkLock` in the [task 
context](../ingestion/tasks.md#context). See [Task Locking & 
Priority](../ingestion/tasks.md#context) for more details about locking in 
tasks.|true|
 |`druid.indexer.tasklock.batchSegmentAllocation`| If set to true, Druid 
performs segment allocate actions in batches to improve throughput and reduce 
the average `task/action/run/time`. See [batching `segmentAllocate` 
actions](../ingestion/tasks.md#batching-segmentallocate-actions) for 
details.|true|
-|`druid.indexer.tasklock.batchAllocationWaitTime`|Number of milliseconds after 
Druid adds the first segment allocate action to a batch, until it executes the 
batch. Allows the batch to add more requests and improve the average segment 
allocation run time. This configuration takes effect only if 
`batchSegmentAllocation` is enabled.|500|
+|`druid.indexer.tasklock.batchAllocationWaitTime`|Number of milliseconds after 
Druid adds the first segment allocate action to a batch, until it executes the 
batch. Allows the batch to add more requests and improve the average segment 
allocation run time. This configuration takes effect only if 
`batchSegmentAllocation` is enabled.|0|
 |`druid.indexer.task.default.context`|Default task context that is applied to 
all tasks submitted to the Overlord. Any default in this config does not 
override neither the context values the user provides nor 
`druid.indexer.tasklock.forceTimeChunkLock`.|empty context|
 |`druid.indexer.queue.maxSize`|Maximum number of active tasks at one 
time.|`Integer.MAX_VALUE`|
 |`druid.indexer.queue.startDelay`|Sleep this long before starting Overlord 
queue management. This can be useful to give a cluster time to re-orient itself 
(for example, after a widespread network issue).|`PT1M`|
diff --git 
a/indexing-service/src/main/java/org/apache/druid/indexing/overlord/config/TaskLockConfig.java
 
b/indexing-service/src/main/java/org/apache/druid/indexing/overlord/config/TaskLockConfig.java
index e750da6c135..2634c4328fe 100644
--- 
a/indexing-service/src/main/java/org/apache/druid/indexing/overlord/config/TaskLockConfig.java
+++ 
b/indexing-service/src/main/java/org/apache/druid/indexing/overlord/config/TaskLockConfig.java
@@ -34,7 +34,7 @@ public class TaskLockConfig
   private boolean batchSegmentAllocation = true;
 
   @JsonProperty
-  private long batchAllocationWaitTime = 500L;
+  private long batchAllocationWaitTime = 0L;
 
   public boolean isForceTimeChunkLock()
   {


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to