Skip to content

Commit

Permalink
Merge pull request #1045 from dpaferov/patch-1
Browse files Browse the repository at this point in the history
Fix a comment which is a bit confusing
  • Loading branch information
mattrjacobs committed Jan 12, 2016
2 parents 944cf45 + 94802e4 commit 90c83c7
Showing 1 changed file with 2 additions and 2 deletions.
Expand Up @@ -42,7 +42,7 @@ public abstract class HystrixCommandProperties {
private static final Integer default_metricsRollingStatisticalWindowBuckets = 10;// default => statisticalWindowBuckets: 10 = 10 buckets in a 10 second window so each bucket is 1 second
private static final Integer default_circuitBreakerRequestVolumeThreshold = 20;// default => statisticalWindowVolumeThreshold: 20 requests in 10 seconds must occur before statistics matter
private static final Integer default_circuitBreakerSleepWindowInMilliseconds = 5000;// default => sleepWindow: 5000 = 5 seconds that we will sleep before trying again after tripping the circuit
private static final Integer default_circuitBreakerErrorThresholdPercentage = 50;// default => errorThresholdPercentage = 50 = if 50%+ of requests in 10 seconds are failures or latent when we will trip the circuit
private static final Integer default_circuitBreakerErrorThresholdPercentage = 50;// default => errorThresholdPercentage = 50 = if 50%+ of requests in 10 seconds are failures or latent then we will trip the circuit
private static final Boolean default_circuitBreakerForceOpen = false;// default => forceCircuitOpen = false (we want to allow traffic)
/* package */ static final Boolean default_circuitBreakerForceClosed = false;// default => ignoreErrors = false
private static final Integer default_executionTimeoutInMilliseconds = 1000; // default => executionTimeoutInMilliseconds: 1000 = 1 second
Expand Down Expand Up @@ -762,4 +762,4 @@ public Setter withRequestLogEnabled(boolean value) {
}


}
}

0 comments on commit 90c83c7

Please sign in to comment.