envoyproxy / envoyproxy/nighthawk

Consider consolidating IsConverged and IsDoomed in Adaptive Load StepController

Open
#466 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
414
Forks
95
Avg merge
2d 9h
Merged PRs (30d)
11

Description

Both are const methods that can change value from the same events.

- We might think of other predicates.
- It could also allow us to encode constraints like doomed implying not converged.
- It may simplify testing.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.