DO NOT REPLY [Bug 36802] - Embeded Tomcat Initialises Context Twice

2005-09-24 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=36802.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36802





--- Additional Comments From [EMAIL PROTECTED]  2005-09-25 00:31 ---
In 5.5.9, the call to init from addChild only happens if you are deploying the 
Context into a Host that has already been started.  In that case, your patch 
doesn't do anything useful since the 'initialized' variable still won't be set 
on the second call to init.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 36802] - Embeded Tomcat Initialises Context Twice

2005-09-24 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=36802.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36802





--- Additional Comments From [EMAIL PROTECTED]  2005-09-25 01:28 ---
Thank you for your comment.

The host has already been started by the time the web application is deployed.

The getParent method constructed the correct ObjectName for the host and it was
found correctly.

Yes for the second call initialized will not be set, however the first call will
be blocked on the invoke on line 5005.

The second call will complete and control will return to the first, initialized
will now be true so the initialisation will not happen again (for the first 
call).


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 36802] - Embeded Tomcat Initialises Context Twice

2005-09-24 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=36802.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36802


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-09-25 02:33 ---
Ok, I see now.  I was thinking that you were trying to stop init being called 
twice (which doesn't seem possible with the current JMX-deployment scheme).

You're patch is still sending the JMX Notification twice, so by the time you 
add that, it seemed much simplier to simply do:
  if( initialized ) {
  return;
  }
So that is what got committed to the CVS, and will appear in 5.5.13.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]