Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only set breaker when stats are retrieved #7721

Merged
merged 1 commit into from Sep 15, 2014

Conversation

dakrone
Copy link
Member

@dakrone dakrone commented Sep 15, 2014

When communicating with 1.3 and earlier nodes, it's possible that the
field data breaker info is not sent at all. When this happens, we should
leave the breaker variable as-is (unset) instead of creating an
AllCircuitBreakerStats object with a null fd breaker and fake request &
parent breakers.

@dakrone
Copy link
Member Author

dakrone commented Sep 15, 2014

Added a BWC test to this.

@s1monw
Copy link
Contributor

s1monw commented Sep 15, 2014

LGTM

@dakrone dakrone force-pushed the fix-breaker-nodes-stats branch 2 times, most recently from 742cfaa to 964db64 Compare September 15, 2014 10:28
When communicating with 1.3 and earlier nodes, it's possible that the
field data breaker info is not sent at all. When this happens, we should
leave the `breaker` variable as-is (unset) instead of creating an
AllCircuitBreakerStats object with a null fd breaker and fake request &
parent breakers.
@dakrone dakrone removed the review label Sep 15, 2014
@dakrone dakrone merged commit 964db64 into elastic:master Sep 15, 2014
@dakrone dakrone deleted the fix-breaker-nodes-stats branch September 15, 2014 10:47
@clintongormley clintongormley changed the title Only set breaker when stats are retrieved Circuit Breaker: Only set breaker when stats are retrieved Sep 26, 2014
@clintongormley clintongormley added the :Core/Infra/Circuit Breakers Track estimates of memory consumption to prevent overload label Mar 19, 2015
@clintongormley clintongormley changed the title Circuit Breaker: Only set breaker when stats are retrieved Only set breaker when stats are retrieved Jun 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Core/Infra/Circuit Breakers Track estimates of memory consumption to prevent overload v1.4.0.Beta1 v1.5.0 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants