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

 ##########
 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)|
-|`targetCompactionSizeBytes`|The target segment size of compaction. The actual 
size of a compact segment might be slightly larger or smaller than this 
value.|no (default = 419430400)|
+|`targetCompactionSizeBytes`|The target segment size after compaction. The 
actual size of a compact segment might be slightly larger or smaller than this 
value.|no (default = 419430400)|
 |`maxNumSegmentsToCompact`|Max number of segments to compact together.|no 
(default = 150)|
 
 Review comment:
   How about:
   
   > Maximum number of segments to compact together per compaction task. Since 
a time chunk must be processed in its entirety, if a time chunk has a total 
number of segments greater than this parameter, compaction will not run for 
that time chunk.

----------------------------------------------------------------
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