dolavb

Executed some more load testing, and we have host spending 60% our total CPU usage on starting threads. This shows the latency resulting from starting thread, 5 times the time...

@atakavci Yes, I think the API considerations are important. It is unfortunate that this now have an API that is essentially a global variable in the static field. Ohterwise, I...

@atakavci I agree there are a couple of ways to do it without adding a breaking change, but I believe the complexity is not worth it. But your call.

@ggivo I agree this is a better approach. @atakavci upon review of your gist I have the following question, why the countdownlatch? Since the introduction of CompletableFuture (JDK8), future composition...

While working on this I saw [this](https://github.com/redis/jedis/pull/2731/files#r1966418712). Do I understand correctly that we shutdown the topologyRefreshExecutor in the JedisClusterInfoCache whenever we close a pipeline? _I apologize for the lack of...

As part of this PR, should we mark `public static volatile int MULTI_NODE_PIPELINE_SYNC_WORKERS = 3;` as deprecated? With a deprecation comment like this: ``` * @deprecated Client using this approach...

@ggivo This makes for a bigger change but offer a path away from the current static implementation. Let me know what you think.

Ultimately I don't mind. We have had that forked and rolled out and is working well with performance issues elsewhere in Redis. On 1, I agree, if the previous approach...