gianm commented on a change in pull request #6782: Improve doc for auto 
compaction
URL: https://github.com/apache/incubator-druid/pull/6782#discussion_r246172502
 
 

 ##########
 File path: docs/content/configuration/index.md
 ##########
 @@ -806,7 +806,7 @@ A description of the compaction config is:
 |`keepSegmentGranularity`|Set 
[keepSegmentGranularity](../ingestion/compaction.html) to true for 
compactionTask.|no (default = true)|
 |`taskPriority`|[Priority](../ingestion/tasks.html#task-priorities) of compact 
task.|no (default = 25)|
 |`inputSegmentSizeBytes`|Total input segment size of a compactionTask.|no 
(default = 419430400)|
 
 Review comment:
   How about:
   
   > Maximum number of total segment bytes processed per compaction task. Since 
a time chunk must be processed in its entirety, if the segments for a 
particular time chunk have a total size in bytes greater than this parameter, 
compaction will not run for that time chunk. Because each compaction task runs 
with a single thread, setting this value too far above 1–2GB will result in 
compaction tasks taking an excessive amount of time.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to