sashidhar commented on issue #8038: Making optimal usage of multiple segment cache locations URL: https://github.com/apache/incubator-druid/pull/8038#issuecomment-514881236 Hi David, Sorry for the unclear wording. It should have been "This PR will introduce...". I was waiting for the design approval before implementing Least-Bytes-Used strategy. I would like to implement both and make the strategy configurable default being Least-Bytes-Used. I'll resume working on the implementation. Thanks, Sashi On Thu, Jul 25, 2019, 5:34 AM David Lim <[email protected]> wrote: > hey @sashidhar <https://github.com/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. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/incubator-druid/pull/8038?email_source=notifications&email_token=AACGIEKI3L6FN7K5Q5EFSDDQBDUY3A5CNFSM4H6T5OQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2X6FEQ#issuecomment-514843282>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AACGIELQDFBIGA3NCQLR3HLQBDUY3ANCNFSM4H6T5OQQ> > . >
---------------------------------------------------------------- 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]
