Skip to content

Commit fc08c3b

Browse files
authored
Merge pull request #55 from elandau/feature/gradient_tweak
Default threshold of 500 micros
2 parents 0bfc4a8 + 8e3e42f commit fc08c3b

File tree

1 file changed

+2
-2
lines changed
  • concurrency-limits-core/src/main/java/com/netflix/concurrency/limits/limiter

1 file changed

+2
-2
lines changed

concurrency-limits-core/src/main/java/com/netflix/concurrency/limits/limiter/DefaultLimiter.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ public final class DefaultLimiter<ContextT> implements Limiter<ContextT> {
2323

2424
private static final long DEFAULT_MIN_WINDOW_TIME = TimeUnit.SECONDS.toNanos(1);
2525
private static final long DEFAULT_MAX_WINDOW_TIME = TimeUnit.SECONDS.toNanos(1);
26-
private static final long DEFAULT_MIN_RTT_THRESHOLD = TimeUnit.MICROSECONDS.toNanos(100);
26+
private static final long DEFAULT_MIN_RTT_THRESHOLD = TimeUnit.MICROSECONDS.toNanos(500);
2727

2828
/**
2929
* Minimum observed samples to filter out sample windows with not enough significant samples
@@ -143,7 +143,7 @@ private DefaultLimiter(Builder builder, Strategy<ContextT> strategy) {
143143
this.limit = builder.limit;
144144
this.minWindowTime = builder.minWindowTime;
145145
this.maxWindowTime = builder.maxWindowTime;
146-
this.minRttThreshold = DEFAULT_MIN_RTT_THRESHOLD;
146+
this.minRttThreshold = builder.minRttThreshold;
147147
this.windowSize = builder.windowSize;
148148
this.strategy = strategy;
149149
strategy.setLimit(limit.getLimit());

0 commit comments

Comments
 (0)