VegasLimit without slow start is unusable about concurrency-limits

[jonomacd]

I'm also looking at a Go implementation and am a bit worried about the slow start problem.

I've noticed the smoothing function here:
https://github.com/Netflix/concurrency-limits/blob/master/concurrency-limits-core/src/main/java/com/netflix/concurrency/limits/limit/VegasLimit.java#L174

Seems to wash out all changes. The limit is only ever incr or decr by 1 so the int truncation on this smoothing seems to keep it at the same value all the time. trunc(0.8n + 0.2(n+1)) = n. I must be missing something here.

[elandau]

@jonomacd There's is a type casting bug in VegasLimit that's messing with updates to the limit for anything other than smoothing=1.0. I've got a fix for it here #36.

Agreed on not slow start being an issue. I can add that quickly once the Vegas smoothing fix is merged.