murblanc commented on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-684067188
> Let us take a step back and reassess at what we have today. ... > There is already an interface called `AssignStrategy` .Can you believe that it's as simple as ... > My suggestion is to not build anything new . Just use this interface You certainly noted in the PR the placement plugins are already doing what you suggest here: just using this interface. Like `LEGACY`, `RULES` (and in 8x `POLICY`) are. This interface is not touched by this PR. The placement plugins code should be compared (if we want comparisons) to the removed code of Autoscaling. ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
