oscerd commented on PR #29479:
URL: https://github.com/apache/camel/pull/29479#issuecomment-6741682924
Thank you @Croway for the thorough review and reproducer tests!
All four points have been addressed in the latest commit:
1. **Cache EIP**: Reverted automatic `cache:` prefixing in `CacheProcessor`.
Cache entries remain un-namespaced, fully preserving the documented
invalidation recipes (`cache.delete(key)` and `cache.clear()`). Added your
invalidation reproducer test in `CacheProcessorTest`.
2. **Backward Compatibility & Dedicated Backends**:
- Added a `namespaced` option (default `true`) on `StateStoreComponent`
and `StateStoreEndpoint`. Users upgrading with existing un-namespaced data on a
shared backend can set `namespaced=false` on the endpoint or component to
access their existing entries without key migration.
- Dedicated backends (created by `StateStoreBackendFactory` or the
default per-store in-memory fallback) are kept un-namespaced.
3. **Prefix Collision Isolation**: Escaped delimiter characters in
`StateStoreComponent.toNamespace` so store names sharing prefixes (e.g.
`orders:archive` vs `orders`) remain strictly isolated. Added your reproducer
test covering this to `StateStoreNamespaceIsolationTest`.
4. **Generated Files**: Regenerated and formatted all catalog and configurer
files.
5. **Documentation**: Updated `state-store-component.adoc` and the upgrade
guide with migration details.
_Antigravity on behalf of oscerd_
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
