You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe
Currently, when creating clustered connections, the adaptive refresh feature is disabled by default.
I've not seen any comments on why this was chosen, but since then almost all examples of usages of Lettuce seem show that adaptive refresh is always enabled by users. It is recommended in default connection settings provided by cloud providers (such as Azure, Google, AWS, Alibaba etc.) and Redis documentation for production usage alike.
Some of the other Redis clients provide adaptive refresh by default and our users have commented that they also expected the same from the Lettuce driver
Describe the solution you'd like
Adaptive refresh was introduced with #240 some 9 years ago.
Changing the defaults would be a breaking change so we should target a major release for that.
We should:
enable all adaptive refresh triggers by default
provide a way for the user to easily disable them or only enable some
mark the enableAllAdaptiveRefreshTriggers() as deprecated to be removed in future releases
Describe alternatives you've considered
Documenting this and increasing visibility.
The problem is that we are still running into clients that are not aware of this feature and are having frequent issues during topology change events.
Feature Request
Is your feature request related to a problem? Please describe
Currently, when creating clustered connections, the adaptive refresh feature is disabled by default.
I've not seen any comments on why this was chosen, but since then almost all examples of usages of Lettuce seem show that adaptive refresh is always enabled by users. It is recommended in default connection settings provided by cloud providers (such as Azure, Google, AWS, Alibaba etc.) and Redis documentation for production usage alike.
Some of the other Redis clients provide adaptive refresh by default and our users have commented that they also expected the same from the Lettuce driver
Describe the solution you'd like
Adaptive refresh was introduced with #240 some 9 years ago.
Changing the defaults would be a breaking change so we should target a major release for that.
We should:
enableAllAdaptiveRefreshTriggers()
as deprecated to be removed in future releasesDescribe alternatives you've considered
Documenting this and increasing visibility.
The problem is that we are still running into clients that are not aware of this feature and are having frequent issues during topology change events.
Teachability, Documentation, Adoption, Migration Strategy
Existing users that depend on this feature being disabled would have to manually disable it.
The text was updated successfully, but these errors were encountered: