janhoy commented on code in PR #1871:
URL: https://github.com/apache/solr/pull/1871#discussion_r1315169354


##########
solr/solr-ref-guide/modules/deployment-guide/pages/circuit-breakers.adoc:
##########
@@ -72,6 +73,30 @@ To enable and configure the CPU utilization based circuit 
breaker:
 
 The `threshold` is defined in units of CPU utilization.
 
+== Advanced example
+
+In this example we will prevent update requests above 80% CPU load, and 
prevent search requests above 95% CPU load.
+This would prevent expensive bulk updates from impacting search. Note also the 
support for short-form class name.
+
+[source,xml]
+----
+<config>
+  <circuitBreaker class="solr.CPUCircuitBreaker">
+   <double  name="threshold">80</double>
+   <arr name="requestTypes">
+     <str>update</str>

Review Comment:
   Done. Since we switched to a more generic Enum class, we needed to validate 
supported types separately, so I just documented in ref guide the two supported 
ones.



-- 
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

Reply via email to