ccaominh commented on a change in pull request #8565: Adjust defaults for 
hashed partitioning
URL: https://github.com/apache/incubator-druid/pull/8565#discussion_r326841747
 
 

 ##########
 File path: docs/ingestion/hadoop.md
 ##########
 @@ -337,7 +337,8 @@ The configuration options are:
 |Field|Description|Required|
 |--------|-----------|---------|
 |type|Type of partitionSpec to be used.|"hashed"|
-|targetRowsPerSegment|Target number of rows to include in a partition, should 
be a number that targets segments of 500MB\~1GB.|either this or `numShards`|
+|targetRowsPerSegment|Target number of rows to include in a partition, should 
be a number that targets segments of 500MB\~1GB. Defaults to 5000000 if 
`numShards`:w
 
 Review comment:
   Ah yes — was in wrong mode for `vi` apparently.

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org

Reply via email to