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;

Reply via email to