chatman commented on a change in pull request #1758: URL: https://github.com/apache/lucene-solr/pull/1758#discussion_r473802780
########## File path: solr/core/src/java/org/apache/solr/cluster/events/ClusterSingleton.java ########## @@ -0,0 +1,14 @@ +package org.apache.solr.cluster.events; + +import java.lang.annotation.Retention; +import java.lang.annotation.RetentionPolicy; + +/** + * Intended for {@link org.apache.solr.core.CoreContainer} plugins that should be + * enabled only one instance per cluster. + * <p>Implementation detail: currently these plugins are instantiated on the + * Overseer leader, and closed when the current node loses its leadership.</p> + */ +@Retention(RetentionPolicy.RUNTIME) +public @interface ClusterSingleton { Review comment: @sigram So long as concept of "triggers" is not a first class citizen in solr-core, I'm +1 to the idea. My understanding was that the autoscaling framework will introduce the concept of "triggers" (with no traces in solr-core) and that framework will also have first party trigger impls. Third party triggers can plug into the framework as dependent packages. ---------------------------------------------------------------- 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