[RUNTIME-53] DefaultKernet implementation forgets to pass through STARTING state

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: 3.4.0
• Component/s: CORE
• Assignee: Stephen McConnell
• Reporter: Carel Paradis
• Created: 15/Aug/04 20:51
• Updated: 13/Nov/04 02:17
• Resolved: 13/Nov/04 02:16

Description

In the startup method, the DefaultKernel implementation must call setState(STARTING) just after verifying if the kernel is startable. Otherwise, kernel event listeners will never be aware that the kernel is starting.

The following file must be updated: trunk\runtime\merlin\impl\src\java\org\apache\avalon\merlin\impl\DefaultKernel.java

The following patch must be applied:

// -----BEGIN PATCH
Index: DefaultKernel.java
===================================================================
— DefaultKernel.java (revision 36413)
+++ DefaultKernel.java (working copy)
@@ -172,6 +172,9 @@
synchronized( m_state )
{
if( !isStartable() ) return;
+
+ setState( STARTING );
+
if( getLogger().isDebugEnabled() )
{
getLogger().debug( "application assembly" );

// -----END PATCH