capistrant opened a new pull request #8462: Add 
druid.segmentCache.numBootstrapThreads back to the docs
URL: https://github.com/apache/incubator-druid/pull/8462
 
 
   ### Description
   
   `druid.segmentCache.numBootstrapThreads` was removed from the documentation 
at some point. However, it is still used when set and I don't see any 
deprecation comments in the code, so I think it should be documented properly. 
I can imagine times when a user may want numBootstrapThreads and 
numLoadingThreads to be different. One specifically is when my team is okay 
with loading on more threads at startup than during post-startup when reaching 
out to HDFS deep store because we don't want to pound our production NameNode 
that has many tenants.
   
   Code 
[link](https://github.com/apache/incubator-druid/blob/master/server/src/main/java/org/apache/druid/segment/loading/SegmentLoaderConfig.java#L52)
 for referene
   
   <hr>
   
   This PR has:
   - [X] been self-reviewed.
   - [X] added documentation for new or modified features or behaviors.
   
   <hr>
   
   ##### Key changed/added classes in this PR
    * Not Applicable

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