redis / lettuce

Advanced Java Redis client for thread-safe sync, async, and reactive usage. Supports Cluster, Sentinel, Pipelining, and codecs.
https://lettuce.io
MIT License
5.37k stars 960 forks source link

`DefaultClientResources.dnsResolver(DnsResolver resolver)` should be marked as deprecated. #2854

Closed yfwz100 closed 4 months ago

yfwz100 commented 4 months ago

Just like definitions at its interface:

https://github.com/redis/lettuce/blob/cc9d2401e654810668d533a0637f32c6d3708b33/src/main/java/io/lettuce/core/resource/ClientResources.java#L164-L175

The implementation method should be marked as deprecated to inform users to use the new settings.

https://github.com/redis/lettuce/blob/cc9d2401e654810668d533a0637f32c6d3708b33/src/main/java/io/lettuce/core/resource/DefaultClientResources.java#L452