Correct invalid @CacheConfig snippet in documentation

[Junnyjun]

Original example used @CacheConfig("books"), which leads to a runtime error because the annotation requires cacheNames to be explicitly defined.
This PR updates the annotation to @CacheConfig(cacheNames = "books") for correctness.

[sbrannen]

Good catch, @Junnyjun. 👍

The documentation is certainly incorrect, though in all fairness, the rest of the cache-related annotations such as @Cacheable, @CachePut, etc., all have aliased value attributes for the cacheNames.

So, I assume:

1. That was an oversight.
2. The documentation was written under the assumption that all cache-related annotations had a value attribute that aliases cacheNames.

[sbrannen]

the rest of the cache-related annotations such as @Cacheable, @CachePut, etc., all have aliased value attributes for the cacheNames.

To address that, I've add a value attribute in 7.0.

• Introduce value alias for cacheNames in @CacheConfig #35152

[Junnyjun]

Hello @sbrannen,

Thank you for your detailed explanation regarding the cacheNames attribute in @CacheConfig and related caching annotations.

also appreciate you creating and completing PR #35152 to address this issue more fundamentally by introducing value alias for cacheNames in @CacheConfig.

this Pull Request (#35096) is no longer necessary, as the core problem will be resolved by PR #35152, so will proceed to close this PR.

[sbrannen]

Hi @Junnyjun,

We still plan to fix the broken documentation example for 6.2.x.

However, I see that you already closed this PR and removed your branch.

Would you like to try to reopen this PR (or create a new one)?

If not, just let us know, and we'll make the change on 6.2.x for you.

Thanks,

Sam

[sbrannen]

Reopening to address the documentation issue in 6.2.x.

[Junnyjun]

I accidentally deleted branch it by pressing the wrong button, but I quickly recovered it

[Junnyjun]

The PR has been reopened 😅