This is an automated email from the ASF dual-hosted git repository.
davsclaus pushed a commit to branch fix/CAMEL-24879
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/fix/CAMEL-24879 by this push:
new 3207ae6e726b CAMEL-24879: Address review feedback
3207ae6e726b is described below
commit 3207ae6e726b66bdbc46b3b6f015551509535e6d
Author: Claus Ibsen <[email protected]>
AuthorDate: Mon Sep 21 22:38:51 2026 +0200
CAMEL-24879: Address review feedback
Initialise the built-in compile post-processors eagerly as a final field
instead of lazily in postCompile.
Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
Signed-off-by: Claus Ibsen <[email protected]>
---
.../camel/language/groovy/DefaultGroovyScriptCompiler.java | 11 ++++-------
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git
a/components/camel-groovy/src/main/java/org/apache/camel/language/groovy/DefaultGroovyScriptCompiler.java
b/components/camel-groovy/src/main/java/org/apache/camel/language/groovy/DefaultGroovyScriptCompiler.java
index 628742204522..de92be2c30f0 100644
---
a/components/camel-groovy/src/main/java/org/apache/camel/language/groovy/DefaultGroovyScriptCompiler.java
+++
b/components/camel-groovy/src/main/java/org/apache/camel/language/groovy/DefaultGroovyScriptCompiler.java
@@ -71,7 +71,10 @@ public class DefaultGroovyScriptCompiler extends
ServiceSupport
private static final Logger LOG =
LoggerFactory.getLogger(DefaultGroovyScriptCompiler.class);
private GroovyPreCompiledClassLoader groovyPreCompiledClassLoader;
- private List<CompilePostProcessor> defaultPostProcessors;
+ private final List<CompilePostProcessor> defaultPostProcessors = List.of(
+ new TypeConverterCompilePostProcessor(),
+ new EventNotifierCompilePostProcessor(),
+ new BindToRegistryCompilePostProcessor());
private GroovyScriptClassLoader classLoader;
private CamelContext camelContext;
private EventNotifier notifier;
@@ -393,12 +396,6 @@ public class DefaultGroovyScriptCompiler extends
ServiceSupport
}
Collection<CompilePostProcessor> posts =
camelContext.getRegistry().findByType(CompilePostProcessor.class);
if (posts == null || posts.isEmpty()) {
- if (defaultPostProcessors == null) {
- defaultPostProcessors = List.of(
- new TypeConverterCompilePostProcessor(),
- new EventNotifierCompilePostProcessor(),
- new BindToRegistryCompilePostProcessor());
- }
posts = defaultPostProcessors;
}
Object instance = null;