CAMEL-24000: camel-spring-boot - upgrade guide entry for Duration configuration properties#24593
Conversation
…figuration properties Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
gnodet
left a comment
There was a problem hiding this comment.
Clean, accurate upgrade guide entry documenting the java.time.Duration property type changes from camel-spring-boot PR #1843.
Verified:
- All six properties correctly listed: four
longfields inSupervisingRouteControllerConfigurationand twointfields inCamelStartupConditionConfigurationProperties - The claim about
camel.clustered.controller.initial-delaykeeping itsStringtype is correct —ClusteredRouteControllerConfigurationwas not modified for Duration conversion - Backward compatibility note is accurate —
@DurationUnit(ChronoUnit.MILLIS)ensures plain numeric values are still interpreted as milliseconds - AsciiDoc formatting follows existing conventions (heading level, backtick-quoted class names, bullet lists)
This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.
Claude Code on behalf of Guillaume Nodet
…nfig properties; sync apache#24592 drift
|
🧪 CI tested the following changed modules:
🔬 Scalpel shadow comparison — Scalpel: 1 tested, 0 compile-only — current: 0 all testedMaveniverse Scalpel detected 1 affected modules (current approach: 0).
|
Claude Code on behalf of Federico Mariani (@Croway)
Upgrade guide entry for CAMEL-24000: camel-spring-boot binds several route controller and startup condition properties as
java.time.Durationin 4.22 (apache/camel-spring-boot#1843). Plain numeric values keep meaning milliseconds; the entry documents the getter/setter type change for programmatic readers and whycamel.clustered.controller.initial-delaykeeps itsStringtype.Requested during review of apache/camel-spring-boot#1843.
🤖 Generated with Claude Code