Brijesh-Thakkar commented on PR #27115:
URL: https://github.com/apache/camel/pull/27115#issuecomment-5916431023
@davsclaus @oscerd pushed an update addressing the review:
- Mapping entries now carry a resolvedAt timestamp and expire after 7 days
(constant PLATFORM_MAPPING_TTL). Expired, timestamp-less or future-dated
entries are re-resolved and overwritten, so a respin like
3.27.3.1 -> 3.27.4 is picked up within a week, or immediately with --fresh.
- storePlatformMapping writes to a temp file in the same directory and uses
ATOMIC_MOVE (falling back to REPLACE_EXISTING if unsupported). Concurrent
processes are last-writer-wins and a lost entry is only a cache miss.
- camel-jbang-devtools.adoc now describes the mapping cache, the TTL and
--fresh.
- New tests use an injected Clock, including the respin-after-TTL case.
Question on docs: CLAUDE.md says the upgrade guide is for migration notes,
but
the 4.21 guide describes the caching behaviour, so I added a short note there
too. Happy to drop that edit and keep only the devtools page if you prefer.
Also happy to change the TTL value.
--
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]
