Replace deprecated API freemarker.template.Configuration.setTemplateUpdateDelay(int) - #25262
Conversation
freemarker.template.Configuration.setTemplateUpdateDelay(int)
```
* @deprecated Use {@link #setTemplateUpdateDelayMilliseconds(long)}
instead, because the time granularity of this method
* is often misunderstood to be milliseconds.
```
Signed-off-by: Aurélien Pupier <apupier@ibm.com>
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
|
🧪 CI tested the following changed modules:
🔬 Scalpel shadow comparison — Scalpel: 9 tested, 29 compile-only — current: 9 all testedMaveniverse Scalpel detected 38 affected modules (current approach: 9).
|
gnodet
left a comment
There was a problem hiding this comment.
Claude Code on behalf of Guillaume Nodet
Correct and behavior-preserving replacement of the deprecated FreeMarker setTemplateUpdateDelay(int) (seconds) with setTemplateUpdateDelayMilliseconds(long) (milliseconds). The 1000L * templateUpdateDelay conversion preserves existing behavior with no overflow risk. CI passes including the existing FreemarkerContentCacheTest which exercises this code path.
Minor note: the PR title "Replace deprecated API" is quite generic — a more descriptive title like "camel-freemarker: Replace deprecated setTemplateUpdateDelay with millisecond variant" would help when scanning git history, but this is non-blocking for a trivial change.
freemarker.template.Configuration.setTemplateUpdateDelay(int)
Description
Target
mainbranch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTestslocally from root folder and I have committed all auto-generated changes.AI-assisted contributions
Co-authored-bytrailers) and the PR description identifies the AI tool used.