-
Notifications
You must be signed in to change notification settings - Fork 294
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
feature: instrument go-redis v9 #634
Conversation
Sorry for nudging, can you take a look at this again, perhaps I still missed something that prevents this PR from being merged @iamemilio ? |
Sorry, we are working on grinding through our PR backlog. We will get to this soon! |
Hi, thanks for contributing and sorry for the delay reviewing this. We are planning more time in our schedule to get on top of Pull Requests. We had some questions and nits, but overall this looked good and did what we expected it to! |
Co-authored-by: Emilio Garcia <iamemilio@users.noreply.github.com>
Thanks @gunturaf! These changes will be in the next release of the go agent! |
We accidentally merged this into master instead of develop. Attempting to re-open |
Please see #658 |
Links
Details
The go-redis package was just moved into
redis
Github organization so the package was renamed fromgithub.com/go-redis/redis
togithub.com/redis/go-redis
. They also just released new major version (v9) with several breaking changes, notably to the hook system.To ensure smooth upgrade from go-redis v8 to v9, a new integration code is needed.