-
Notifications
You must be signed in to change notification settings - Fork 71
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
optimize fast path with Duration-based wait time #368
optimize fast path with Duration-based wait time #368
Conversation
I've run a comparison of 0.13.3, 0.14.1-dev without this PR, and 0.14.1-dev with this PR, confirming that this seems to fix the reported performance regression: |
I can confirm that this PR fixes the #367. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RTBM - with two nits / questions
0.13.3
0.14.1-dev (before this PR)
0.14.1-dev (after this PR)