Skip to content

Conversation

@kanderson250
Copy link
Contributor

Resolves #2119

This is a pure lift of the existing lettuce instrumentation, so that it now applies for lettuce versions 6.5.0-RELEASE and above.

Changes:

  • Replaces calls to ProtocolKeyword.name() with ProtocolKeyword.toString() per the lettuce API
  • Removes obsolete constructor in StatefulRedisConnectionImpl_Instrumentation
  • Includes recent update to use the redis DB index when a redis URI is available

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.11%. Comparing base (1729b78) to head (c9b3b51).
Report is 100 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2430      +/-   ##
============================================
- Coverage     70.60%   69.11%   -1.49%     
+ Complexity    10223     9859     -364     
============================================
  Files           844      845       +1     
  Lines         41179    40693     -486     
  Branches       6333     6171     -162     
============================================
- Hits          29073    28126     -947     
- Misses         9286     9823     +537     
+ Partials       2820     2744      -76     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kanderson250 kanderson250 merged commit 97bf932 into main Jul 23, 2025
545 of 551 checks passed
@github-project-automation github-project-automation bot moved this from Triage to Code Complete/Done in Java Engineering Board Jul 23, 2025
@kanderson250 kanderson250 deleted the lettuce-6.5 branch July 23, 2025 20:23
@kmudduluru kmudduluru moved this to Code Complete/Done in Java Engineering Board Jul 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Add new instrumentation for io.lettuce:lettuce-core:6.5.0.RELEASE and above

4 participants