You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@solr.apache.org by "Ilan Ginzburg (Jira)" <ji...@apache.org> on 2021/11/25 02:04:00 UTC
[jira] [Comment Edited] (SOLR-15209) Make the AffinityPlacementFactory the default placement plugin
[ https://issues.apache.org/jira/browse/SOLR-15209?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17448900#comment-17448900 ]
Ilan Ginzburg edited comment on SOLR-15209 at 11/25/21, 2:03 AM:
-----------------------------------------------------------------
This is vastly more time consuming than I expected. Switching all tests to a new placement strategy does break many of them. Most obvious tests breaking are those trying to place more replicas of a shard than there are nodes (the placement plugin in its current form does not allow that).
Current state of the work is in branch [SOLR-15209_placementDefault|https://github.com/murblanc/solr/tree/SOLR-15209_placementDefault] on my github (2 last commits as of now).
I will pause that work for now, I don't have the time.
If somebody wants to pick from where I left, please... and I can help.
I focused on tests failures. I did not test the actual new configuration in {{solr.xml}} allowing to go back to legacy assign strategy, and I did not test running a cluster "out of the box" and verifying the plugin affinity assign strategy works as expected. There are too many test failures anyway at this stage.
was (Author: murblanc):
This is vastly more time consuming than I expected. Switching all tests to a new placement strategy does break many of them. Most obvious tests breaking are those trying to place more replicas of a shard than there are nodes (the placement plugin in its current form does not allow that).
Current state of the work is in branch [SOLR-15209_placementDefault|https://github.com/murblanc/solr/tree/SOLR-15209_placementDefault] on my github.
I will pause that work for now, I don't have the time.
If somebody wants to pick from where I left, please... and I can help.
I focused on tests failures. I did not test the actual new configuration in {{solr.xml}} allowing to go back to legacy assign strategy, and I did not test running a cluster "out of the box" and verifying the plugin affinity assign strategy works as expected. There are too many test failures anyway at this stage.
> Make the AffinityPlacementFactory the default placement plugin
> --------------------------------------------------------------
>
> Key: SOLR-15209
> URL: https://issues.apache.org/jira/browse/SOLR-15209
> Project: Solr
> Issue Type: Improvement
> Reporter: Andrzej Bialecki
> Assignee: Ilan Ginzburg
> Priority: Blocker
> Fix For: main (9.0)
>
>
> Currently there's a lot of code in {{Assign}} dealing with the fact that we still support the old "legacy" replica assignment as well as the new plugin-based placement strategies.
> Furthermore, the "legacy" assignment is now the default even though it's neither robust nor optimal, except in the very simple and small clusters. Also, providing another plugin-based placement as the default impl. runs into a small complication - in the absence of any plugin config the code reverts to the "legacy".
> In order to promote the adoption of the new plugin-based placements we should make the {{AffinityPlacementFactory}} the new default placement strategy, selected when the explicit configuration is missing (and then create it as a default configuration in {{PlacementPluginFactoryLoader}}).
> I propose to re-package the "legacy" strategy as a {{PlacementPluginFactory}} so that it can be configured in the same way as other placement plugins.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org