No, it's bad. You don't follow with DRY & KISS.
1. These properties have to be moved to the `AbstractScriptExecutingMessageProcessor`
2. There is no reason to do any change in the `ScriptExecutingMessageProcessor`
3. `GroovyScriptExecutingMessageProcessor` has to implement `InitializingBean` to enrich `GroovyScriptFactory` with null checks, of course.
4. `GroovyCommandMessageProcessor` isn't registered as a bean: it is a product of `GroovyControlBusFactoryBean`. So, the last one has to `implements BeanFactoryAware, BeanClassLoaderAware` too and delegates `beanClassLoader` & `beanFactory` to the `GroovyCommandMessageProcessor`.

Sorry, if I am so picky :smile: 
Thanks

P.S. 	indents should be tabs, not whitespaces.