Support lettuce latency metrics

[racevedoo]

Hi all!

Are there any plans to support lettuce command latency metrics? Shouldn't be too hard to implement using micrometer.

[ilopmar]

Not yet, but of course PRs welcome :)

[racevedoo]

Sure! I'll try to come up with a draft

[racevedoo]

@ilopmar seems like lettuce 6.0 changed the way to collect metrics a bit, as stated here.

I see that there's an open PR that bumps the lettuce version to 6.0.1, so I'd like to start implementing this using 6.0 already. Is there any problem? How should I proceed? Thanks!!

[graemerocher]

@ilopmar we will need to bump the major version of the project

[ilopmar]

Before we can bump a major version of a dependency and include it in a minor Micronaut version, we need to make sure that there are no breaking changes in that module. In case there are breaking changes we need to think carefully if they will accept users' applications and how. I see that the PR with the upgrade pass all the test, that's a good start :-)

Another thing that we need to test before the upgrade is the GraalVM integration. We already have a test application so we need to use it and make sure everything still works.

Yes, you can start working on that and we could include a major version of this module in Micronaut 2.3.

[ilopmar]

Upgraded to Lettuce 6.0.2 in #123 and released in Micronaut Redis 4.0.0

[racevedoo]

@ilopmar do we support latency metrics with the upgrade? I guess that was the original issue

[ilopmar]

No, there is nothing built-it for doing that. I though it was only a matter of upgrading to 6.0.