⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

warn log only if cache feature is enabled #1637

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
ryanjbaxter merged 2 commits into from
Jan 17, 2026
Merged

warn log only if cache feature is enabled #1637

ryanjbaxter merged 2 commits into from
Jan 17, 2026
+20 −26

Conversation

@B18150228NJUPT
Copy link
Contributor

@B18150228NJUPT B18150228NJUPT commented Jan 15, 2026

two problems:

  1. LoadBalancerCaffeineWarnLogger print warn log even if cache configuration is not configured, this makes developer confused
  2. warn log shows that framework uses default cache, but default cache is enabled when evictor dependency is added, but by default this dependency is absent

@github-project-automation github-project-automation bot moved this to Todo in 2025.1.1 Jan 17, 2026
@ryanjbaxter ryanjbaxter added this to the 5.0.1 milestone Jan 17, 2026
@ryanjbaxter ryanjbaxter merged commit 9f0e1d4 into spring-cloud:main Jan 17, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in 2025.1.1 Jan 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ryanjbaxter ryanjbaxter ryanjbaxter left review comments

Assignees

No one assigned

Labels

task

Projects

2025.1.1
Status: Done

Milestone

5.0.1

Development

Successfully merging this pull request may close these issues.

3 participants

@B18150228NJUPT @ryanjbaxter @spring-cloud-issues