-
Notifications
You must be signed in to change notification settings - Fork 0
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
Upgrade go-redis #41
Upgrade go-redis #41
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #41 +/- ##
=======================================
Coverage 73.52% 73.52%
=======================================
Files 2 2
Lines 34 34
=======================================
Hits 25 25
Misses 7 7
Partials 2 2
Continue to review full report in Codecov by Sentry.
|
0fa686a
to
70fe11d
Compare
``` name old time/op new time/op delta _CloudflareAlgorithm-10 28.4µs ± 1% 26.4µs ± 0% -7.07% (p=0.008 n=5+5) _GcraAlgorithm-10 31.6µs ± 0% 31.4µs ± 1% -0.76% (p=0.032 n=5+5) _SlidingWindowAlgorithm-10 33.0µs ± 2% 32.9µs ± 1% ~ (p=1.000 n=5+5) name old alloc/op new alloc/op delta _CloudflareAlgorithm-10 1.29kB ± 0% 0.82kB ± 1% -36.75% (p=0.000 n=4+5) _GcraAlgorithm-10 831B ± 0% 831B ± 0% ~ (p=0.333 n=4+5) _SlidingWindowAlgorithm-10 640B ± 0% 640B ± 0% ~ (all equal) name old allocs/op new allocs/op delta _CloudflareAlgorithm-10 37.0 ± 0% 25.0 ± 0% -32.43% (p=0.008 n=5+5) _GcraAlgorithm-10 26.0 ± 0% 26.0 ± 0% ~ (all equal) _SlidingWindowAlgorithm-10 21.0 ± 0% 21.0 ± 0% ~ (all equal) ```
EvalRO(ctx context.Context, script string, keys []string, args ...interface{}) *redis.Cmd | ||
EvalShaRO(ctx context.Context, sha1 string, keys []string, args ...interface{}) *redis.Cmd |
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.
Those are new?
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.
Yes, needed for upgraded packages.
Comparing to master:
Moving to MGET instead of pipeline for cloudflare: