janhoy commented on code in PR #1725: URL: https://github.com/apache/solr/pull/1725#discussion_r1304292604
########## solr/solr-ref-guide/modules/deployment-guide/pages/circuit-breakers.adoc: ########## @@ -27,23 +27,16 @@ If circuit breakers are enabled, requests may be rejected under the condition of It is up to the client to handle this error and potentially build a retrial logic as this should ideally be a transient situation. == Circuit Breaker Configurations -All circuit breaker configurations are listed in the `<circuitBreaker>` tags in `solrconfig.xml` as shown below: +All circuit breaker configurations are listed as independent `<circuitBreaker>` entries in `solrconfig.xml` as shown below: [source,xml] ---- -<circuitBreaker class="solr.CircuitBreakerManager" enabled="true"> - <!-- All specific configs in this section --> +<circuitBreaker class="org.apache.solr.util.circuitbreaker.CPUCircuitBreaker"> + <int name="threshold">75</int> </circuitBreaker> ---- -The `enabled` attribute controls the global activation/deactivation of circuit breakers. -If this flag is disabled, all circuit breakers will be disabled globally. -Per circuit breaker configurations are specified in their respective sections later. - -This attribute acts as the highest authority and global controller of circuit breakers. -For using specific circuit breakers, each one needs to be individually enabled in addition to this flag being enabled. - -`CircuitBreakerManager` is the default manager for all circuit breakers that should be defined in the tag unless the user wishes to use a custom implementation. +Each circuit breaker can be independently enabled/disabled using the corresponding flag. Review Comment: Hmm, will have to go a round with this enable/disable config/logic. Probably no need for the toggle at all, if you can simply not add the plugin in the first place. -- 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. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org