Feature Request: Metrics -or- EventBus Event for EventTimeoutExceptions

Current Version:

        <dependency>
            <groupId>io.lettuce</groupId>
            <artifactId>lettuce-core</artifactId>
            <version>5.0.4.RELEASE</version>
        </dependency>

Wanted to discuss possibly getting support for timeout events on the RedisClient’s event bus or a metric for the CommandMetrics object that stores a count of timeouts (preferring the latter). I’ve looked for this functionality in the lettuce API but have not found it. I would be open to helping collaborate on this feature.

[mp911de]

@gtschurwald Take a lot at https://github.com/lettuce-io/lettuce-core/wiki/Command-Latency-Metrics. Timeouts do not apply to commands but rather to the synchronization/blocking facade.

[gtschurwald]

I’ve had to put this aside for now in order to tackle some more urgent work, if someone else works on adding this that would be fantastic.

For some quick ideas on how to implement this, it seems like the method awaitOrCancel in LettuceFutures might be a good point to increment some kind of failure counter, or the TimeoutException it throws could be caught and trigger a metric increment. Alternatively, perhaps the set timeout could just be compared against the latency metric for completion time when that gets reported to determine a “timeout” event, but I’m not sure if latency metrics get reported on commands that timeout.