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.

Sign up for GitHub

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

Jump to bottom

fix: replace async-timer with futures-timer #324

Merged

geofmureithi merged 1 commit into master from bug/remove-async-timer

May 24, 2024

Owner

geofmureithi commented May 22, 2024 •

edited

Loading

Attempts to resolve #291 and #301 without a version bump


          fix: replace async-timer with futures-timer

5c4d827

github-actions bot commented May 22, 2024

Benchmark for `b384ce2`

Click to view benchmark

Test	Base	PR	%
postgres/consume/1000	5.2±0.00s	5.2±0.00s	0.00%
postgres/push/1000	513.6±6.28ms	513.1±15.97ms	-0.10%
redis/consume/1000	10.1±0.00s	10.1±0.00s	0.00%
redis/push/1000	159.6±1.03ms	160.1±0.81ms	+0.31%
sqlite_in_memory/consume/1000	6.0±0.00s	6.0±0.00s	0.00%
sqlite_in_memory/push/1000	73.2±0.63ms	71.8±0.42ms	-1.91%

Owner Author

geofmureithi commented May 22, 2024

@IgnisDa does this solve your problem?

IgnisDa commented May 24, 2024

@geofmureithi Seems to be working fine locally.

geofmureithi merged commit f63480c into master

5 checks passed

geofmureithi deleted the bug/remove-async-timer branch

May 24, 2024 19:32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet