Skip to content
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

fix: improve external api for redis #325

Closed
wants to merge 3 commits into from
Closed

Conversation

geofmureithi
Copy link
Owner

No description provided.

Copy link

Benchmark for c6bb43e

Click to view benchmark
Test Base PR %
postgres/consume/1000 5.3±0.05s 5.2±0.00s -1.89%
postgres/push/1000 481.7±7.15ms 472.8±10.23ms -1.85%
redis/consume/1000 10.1±0.00s 10.1±0.00s 0.00%
redis/push/1000 157.0±1.44ms 154.5±1.63ms -1.59%
sqlite_in_memory/consume/1000 6.0±0.00s 6.0±0.00s 0.00%
sqlite_in_memory/push/1000 70.8±0.79ms 71.5±0.96ms +0.99%

Copy link

Benchmark for c05dcba

Click to view benchmark
Test Base PR %
postgres/consume/1000 5.3±0.05s 5.2±0.05s -1.89%
postgres/push/1000 567.3±9.31ms 526.2±18.18ms -7.24%
redis/consume/1000 10.1±0.00s 10.1±0.00s 0.00%
redis/push/1000 157.0±1.41ms 159.7±1.00ms +1.72%
sqlite_in_memory/consume/1000 6.0±0.00s 6.0±0.03s 0.00%
sqlite_in_memory/push/1000 71.7±0.76ms 70.8±0.82ms -1.26%

@geofmureithi
Copy link
Owner Author

Closed because of #338

@geofmureithi geofmureithi reopened this Jul 4, 2024
@geofmureithi geofmureithi deleted the fix/better-api branch July 4, 2024 18:51
Copy link

github-actions bot commented Jul 4, 2024

Benchmark for f2aae2d

Click to view benchmark
Test Base PR %
postgres/consume/1000 5.2±0.00s 5.2±0.03s 0.00%
postgres/push/1000 509.6±5.97ms 504.9±11.77ms -0.92%
redis/consume/1000 10.1±0.00s 10.1±0.00s 0.00%
redis/push/1000 160.4±0.98ms 159.7±1.82ms -0.44%
sqlite_in_memory/consume/1000 6.0±0.03s 6.0±0.03s 0.00%
sqlite_in_memory/push/1000 71.9±0.59ms 72.3±0.71ms +0.56%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant