[
https://issues.apache.org/jira/browse/SOLR-14588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17145999#comment-17145999
]
Atri Sharma edited comment on SOLR-14588 at 6/26/20, 4:14 AM:
--------------------------------------------------------------
[~erickerickson] Thank you for proactively fixing the test!
Did you get a chance to see the PR I raised late last night?
(https://github.com/apache/lucene-solr/pull/1615)This fixes the same by not
printing circuit breaker timings when circuit breakers are disabled. Would you
feel that this is a longer term strategy to follow since this insulates the
circuit breaker response messages breaking existing tests and only affecting
ones which want to run with circuit breakers enabled?
was (Author: atris):
[~erickerickson] Thank you for proactively fixing the test!
Did you get a chance to see the PR I raised late last night
(https://github.com/apache/lucene-solr/pull/1615)? This fixes the same by not
printing circuit breaker timings when circuit breakers are disabled. Would you
feel that this is a longer term strategy to follow since this insulates the
circuit breaker response messages breaking existing tests and only affecting
ones which want to run with circuit breakers enabled?
> Circuit Breakers Infrastructure and Real JVM Based Circuit Breaker
> ------------------------------------------------------------------
>
> Key: SOLR-14588
> URL: https://issues.apache.org/jira/browse/SOLR-14588
> Project: Solr
> Issue Type: Improvement
> Security Level: Public(Default Security Level. Issues are Public)
> Reporter: Atri Sharma
> Assignee: Atri Sharma
> Priority: Major
> Time Spent: 8h
> Remaining Estimate: 0h
>
> This Jira tracks addition of circuit breakers in the search path and
> implements JVM based circuit breaker which rejects incoming search requests
> if the JVM heap usage exceeds a defined percentage.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
