Tihomir Krasimirov Mateev
Tihomir Krasimirov Mateev
> Hey @thachlp , there is a number of tests that fail after this change, most likely because they do not expect these timeout options to be set by default....
The failing tests are failing in other PRs too, seems to be a regression, investigating currently ...
> The failing tests are failing in other PRs too, seems to be a regression, investigating currently ... You need to fetch the fix from https://github.com/redis/lettuce/pull/2949
TBH I am not sure what we gain from this code. The performance measurements are a bit inconclusive and as far as I can see this improvement only benefits a...
Hey @Roiocam is this related to #2935?
Hey @nits3392 , as of right now unfortunately I cannot commit to a timeline as we are overloaded with a couple of urgent requests.
Hey @okg-cxf , could we take a step back and talk a bit more about the use case you are handling? Can you give a short example of the issue...
@leisurelyrcxf could you help me understand better the scenario you are addressing? If you could provide a stack trace that explains it it would be perfect.
Awesome, thanks for clarifying, let me get back to you after I spend some time thinking about this. Specific implementation aside this seems to be a meaningful thing to fix.
I think this change makes sense. Let's try and get it out with version 6.6