Tihomir Krasimirov Mateev

Results 283 comments of Tihomir Krasimirov Mateev

> Looks like a variant of #2175 Do you think it is more likely a problem of the driver or the way spring-data-redis uses it? I am trying to see...

Hey @zafir-zhong , a few followup questions: - do you see any other issues in the log that precede this one (Out-of-Memory, NullPointerException, etc.)? - do you use a custom...

@RootSir is this still relevant? I am afraid I don't understand the question, could you explain the specific issue you are facing?

> This looks really good! It will be a nice addition to Lettuce. Just a couple things: > > * I did not see tests against Redis Cluster. > *...

> Hey @tishun, `...tracking...` stuff got in here. Is it intended? No, but this is because the PR is fork/redis-json -> origin/redis-json and origin/redis-json is now behind origin/main I will...

> Overall, this is a good first iteration, but binding all JSON interfaces to generic types of Lettuce codec seems unnecessary and just increases the complexity of the code. There...

> > Overall, this is a good first iteration, but binding all JSON interfaces to generic types of Lettuce codec seems unnecessary and just increases the complexity of the code....

Hey @a-khakimov, to the best of my knowledge there are no benchmarks for the current implementation. Having said that - this first implementation that Mark did has some limitations (most...

Hey @jiajunsu , Thanks for the verbose analysis. Before we jump into the way the Lettuce driver works let's first analyse the scenario and environment you are working with and...

Oh, okay, let me address these one at the time: > With 2 sentinel, 1 upstream and 1 replica nodes running. This is [not a recommended setup](https://redis.io/docs/latest/operate/oss_and_stack/management/sentinel/#fundamental-things-to-know-about-sentinel-before-deploying) for Redis Sentinel...