dclim commented on issue #8038: Making optimal usage of multiple segment cache 
locations
URL: https://github.com/apache/incubator-druid/pull/8038#issuecomment-514843282
 
 
   hey @sashidhar - your proposal mentioned:
   
   > This PR introduces an optional new Historical runtime property 
druid.segmentCache.locationSelectorStrategy to make the segment cache location 
selection strategy configurable. Possible values for the above property - 
round-robin, least-bytes-used.
   
   But I don't see that implemented (I only see the round-robin 
implementation). Am I missing something? You also said:
   
   >  It makes sense for the Least-Bytes-Used to be the default for the write 
and read throughput reasons mentioned.
   
   Which I agree with. Apologies if you were waiting on further confirmation 
before implementing the Least-Bytes-Used strategy. Between round-robin and 
Least-Bytes-Used, I would be okay if you just implemented the latter, as I 
think it would be the right option in the majority of cases, but I would also 
be okay if you implemented both and had a configuration parameter to select the 
strategy.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

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

Reply via email to