Add support for Jedis maxTotalRetriesDuration in config

[joshua5201]

Hi team,
I want to use maxTotalRetriesDuraion option in JedisCluster introduced by this Jedis PR.

I created a draft. I don't know if I should use a new configuration class or just use RedisClusterConfiguration since this is a Jedis-only option.

Please kindly give me some directions then I can do further development and testing.

• You have read the Spring Data contribution guidelines.
• You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
• You submit test cases (unit or integration tests) that back your changes.
• You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

[pivotal-cla]

@joshua5201 Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

[pivotal-cla]

@joshua5201 Thank you for signing the Contributor License Agreement!

[mp911de]

Thanks for your pull request. The max-redirect duration isn't supported across all Redis clients and I don't see that Lettuce will ever support such a feature, therefore we cannot expose this configuration item into the ClusterConfiguration respective RedisClusterConfiguration API.

Additionally, Jedis has this bad habit of putting all sorts of arguments into constructors that make it difficult to provide customization hooks. It would be much easier to provide a Jedis-specific option if all cluster configuration would be encapsulated, something along the lines of the already existing JedisClientConfig design.

In its current form, we cannot accept your contribution because we don't have a good way to make it work.

As a workaround, please subclass JedisConnectionFactory and override the createCluster(…) method.