You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by "Forrest Xia (JIRA)" <ji...@apache.org> on 2009/02/22 10:13:02 UTC

[jira] Created: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
------------------------------------------------------------------------------------

                 Key: GERONIMO-4549
                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
             Project: Geronimo
          Issue Type: Bug
      Security Level: public (Regular issues)
          Components: ActiveMQ
    Affects Versions: 2.1.4
            Reporter: Forrest Xia
            Priority: Minor


Steps to reproduce this problem:
1. login admin console
2. Create a ActiveMQ resource connector with the wizard
3. Deploy it and check it is in running state
4. Click J2EE connector to uninstall it
5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Forrest Xia (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Forrest Xia reopened GERONIMO-4549:
-----------------------------------


> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.5, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jack Cai (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12678767#action_12678767 ] 

caijunj edited comment on GERONIMO-4549 at 3/4/09 8:10 AM:
------------------------------------------------------------

So I spend more time to understand the code. Guess I now know the reason why Context A is chosen that way (to quickly remove a chain of subcontext which only contains the object to be removed). But I still don't get why removeNotEmptyContext is set to false in org.apache.xbean.naming.context.WritableContext$NestedWritableContext.removeBinding().

I propose 2 fixes as - 

1. Update org.apache.xbean.naming.context.WritableContext$NestedWritableContext.removeBinding(), new code below (change false to removeNotEmptyContext)
{code}
        protected boolean removeBinding(String name, boolean removeNotEmptyContext) throws NamingException {
            if (WritableContext.this.removeBinding(bindingsRef, name, removeNotEmptyContext)) {
                return true;
            }
            return super.removeBinding(name, removeNotEmptyContext);
        }
{code}

2. Update the org.apache.xbean.naming.context.AbstractContext.removeDeepBinding() to remove the chain of subcontexts one by one starting from the leaf subcontexts, so that each subcontext IS empty when being removed and can thus be removed successfully even if removeNotEmptyContext is set to false.

Any comments? I'll submit the patch tomorrow. But I'd like to get some advice beforehands.


      was (Author: caijunj):
    So I spend more time to understand the code. Guess I now know the reason why Context A is chosen that way (to quickly remove a chain of subcontext which only contains the object to be removed). But I still don't get why removeNotEmptyContext is set to false in org.apache.xbean.naming.context.WritableContext$NestedWritableContext.removeBinding().

I propose 2 fixes as - 

1. Update org.apache.xbean.naming.context.WritableContext$NestedWritableContext.removeBinding(), new code below (change false to removeNotEmptyContext)
{code}
        protected boolean removeBinding(String name, boolean removeNotEmptyContext) throws NamingException {
            if (WritableContext.this.removeBinding(bindingsRef, name, removeNotEmptyContext)) {
                return true;
            }
            return super.removeBinding(name, removeNotEmptyContext);
        }
{/code}

2. Update the org.apache.xbean.naming.context.AbstractContext.removeDeepBinding() to remove the chain of subcontexts one by one starting from the leaf subcontexts, so that each subcontext IS empty when being removed and can thus be removed successfully even if removeNotEmptyContext is set to false.

Any comments? I'll submit the patch tomorrow. But I'd like to get some advice beforehands.

  
> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jack Cai (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12678767#action_12678767 ] 

Jack Cai commented on GERONIMO-4549:
------------------------------------

So I spend more time to understand the code. Guess I now know the reason why Context A is chosen that way (to quickly remove a chain of subcontext which only contains the object to be removed). But I still don't get why removeNotEmptyContext is set to false in org.apache.xbean.naming.context.WritableContext$NestedWritableContext.removeBinding().

I propose 2 fixes as - 

1. Update org.apache.xbean.naming.context.WritableContext$NestedWritableContext.removeBinding(), new code below (change false to removeNotEmptyContext)
{code}
        protected boolean removeBinding(String name, boolean removeNotEmptyContext) throws NamingException {
            if (WritableContext.this.removeBinding(bindingsRef, name, removeNotEmptyContext)) {
                return true;
            }
            return super.removeBinding(name, removeNotEmptyContext);
        }
{/code}

2. Update the org.apache.xbean.naming.context.AbstractContext.removeDeepBinding() to remove the chain of subcontexts one by one starting from the leaf subcontexts, so that each subcontext IS empty when being removed and can thus be removed successfully even if removeNotEmptyContext is set to false.

Any comments? I'll submit the patch tomorrow. But I'd like to get some advice beforehands.


> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Donald Woods (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Donald Woods updated GERONIMO-4549:
-----------------------------------

         Priority: Blocker  (was: Minor)
    Fix Version/s: 2.2
                   2.1.4

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jarek Gawor (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jarek Gawor updated GERONIMO-4549:
----------------------------------

    Fix Version/s:     (was: 2.1.4)
                   2.1.5

Updated affects/fix versions as this won't get fixed in time for 2.1.4.


> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.5, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "David Jencks (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679081#action_12679081 ] 

David Jencks commented on GERONIMO-4549:
----------------------------------------

Jack,

I don't completely understand the xbean jndi code however I think the reason for some of the things like removeNotEmptyContext are to support federated jndi contexts.  So I recommend you try to understand federated jndi contexts better than I do before changing the code... or come up with more unit tests.


> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "David Jencks (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Jencks closed GERONIMO-4549.
----------------------------------

    Resolution: Fixed

Assuming this is actually fixed in xbean... trunk has been using xbean snapshots for a long time.  Rev 803714 makes the 2.1 branch use xbean snapshots.  I didn't see any obvious problems.



> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.5, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Joe Bohn (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679693#action_12679693 ] 

Joe Bohn commented on GERONIMO-4549:
------------------------------------

So it seems that this really is an XBean issue.  We should create a JIRA under their project: http://issues.apache.org/jira/browse/XBEAN
For us to get this into Geronimo 2.1.4 we would need another xbean release first.  It would most likely delay 2.1.4 more than just a few days.

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.4, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Joe Bohn (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679222#action_12679222 ] 

Joe Bohn commented on GERONIMO-4549:
------------------------------------

I see no trace of the name anywhere after I uninstall - even under Global context.  Are you sure you are seeing this problem with the latest 2.1.4-SNAPSHOT?  Either it's been resolved or there is some nuance to the scenario that I'm not hitting (which might help narrow down the cause and proposed fix a bit more).

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Joe Bohn (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679241#action_12679241 ] 

Joe Bohn commented on GERONIMO-4549:
------------------------------------

Ok, I see the failure now.  You must add a destination in addition to a connection factory.  Adding a connection factory alone to the resource group and all still works as expected on uninstall - the connection factory jndi name *is* removed from the global context.  However, if you add both a connection factory and a destination into the same plan and deploy it both the connection factory and destination will remain under global context even after uninstall.  

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jack Cai (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jack Cai reassigned GERONIMO-4549:
----------------------------------

    Assignee: Jack Cai

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Forrest Xia (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12693989#action_12693989 ] 

Forrest Xia commented on GERONIMO-4549:
---------------------------------------

Until we merge in the XBean fix to g, I don't think we can close this bug now. 

Leave it open, so that we can filter it out some day in future and verify it if XBean patch is merged.

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.5, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jack Cai (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12678007#action_12678007 ] 

Jack Cai commented on GERONIMO-4549:
------------------------------------

I took quite some time to look into this problem since it's marked as a blocker and we are approching 2.1.4 release. I've nailed the problem down to the xbean-naming code, but I'm not sure how to fix it. Would like to get other's advice. Below is the summary of my findings -

1. The sympton is that when a connection factory and a queue are created and started, and then stopped/uninstalled, their JNDI entry will continue to exist in the global context unless the server is restarted.

2. When stoping/uninstalling them, org.apache.xbean.naming.context.AbstractContext.removeDeepBinding() will eventually be called, which finds the last context whose size > 2 in the context chain, then tries to remove the entry from this context (say Context A). This results in calling org.apache.xbean.naming.context.WritableContext$NestedWritableContext.removeBinding(), which uses removeNotEmptyContext=false to do removal. Unfortunately the context (say Context B) being removed (in Context A) is always not empty, resulting a ContextNotEmptyException which is ignored in org.apache.geronimo.gjndi.KernelContextGBean.removeBinding().

I don't have enough background to understand why Context A is chosen this way and why removeNotEmptyContext is set to false. Can anyone shed some light here?

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jack Cai (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jack Cai updated GERONIMO-4549:
-------------------------------

    Attachment: Geronimo-4549_0307.patch

I've got some progress now.

1. For the original problem, it happens in such scenario: if there are two names which share some common intermediate parts, e.g., "test/test1/GBean/R1" and "test/test2/GBean/R2", then any attempts to remove any of them will fail. This can be fixed by using passed-in removeNotEmptyContext instead of "false" in org.apache.xbean.naming.context.WritableContext$NestedWritableContext.removeBinding(). I've read through related code and feel it's a safe update.

2. There is another more serious problem exposed after the above problem is fixed, i.e., the index entries in WritableContext will not be removed when the corresponding object is removed. This will result in subsequent "lookup" for that object continuing to succeed. I've also fixed this.

I wrote two testcases for future regression. The patched code passes all previous unit tests in addition to the new ones. Also verified the JMS resource issue in Admin Console is now resolved.

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.4, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Joe Bohn (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Joe Bohn updated GERONIMO-4549:
-------------------------------

             Priority: Minor  (was: Blocker)
    Affects Version/s: 2.2
                       2.1.1
                       2.1.2
                       2.1.3

I moved this from blocker back to minor.  I saw no justification for why it was made a blocking issue.  I also verified the same issue is present in 2.1.1, 2.1.2, and 2.1.3.

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jack Cai (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jack Cai closed GERONIMO-4549.
------------------------------

    Resolution: Fixed

Closing this issue as it's a XBean issue.

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.5, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Donald Woods (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679683#action_12679683 ] 

Donald Woods commented on GERONIMO-4549:
----------------------------------------

Joe, +1 for getting this into 2.1.4, even if it delays a RC by a couple days....


> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.4, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Joe Bohn (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679405#action_12679405 ] 

Joe Bohn commented on GERONIMO-4549:
------------------------------------

BTW, one other interesting tidbit.  Once you have at least one jndi resource entry that is not removed subsequent entries will likewise not be removed (even if you then deploy and uninstall a resource group with just a connector). 

For example, if you deploy and undeploy a resource group with just one connector and no destination the connector entry will be removed from the jndi global context.
If you deploy a resource group with both a connector and destination the global context entries will not be removed on uninstall.  
And, if you subsequently deploy and undeploy another resource group with just a connector (no destination) after the failure to cleanup mentioned above the new connector will also continue to exist in global context after the uninstall.


> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jack Cai (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679822#action_12679822 ] 

Jack Cai commented on GERONIMO-4549:
------------------------------------

If pulling in this fix solely delays Geronimo 2.1.4 release, then I'm inclined to exclude it for the time being. After all, it's almost 6 months now since last Geronimo release. Let's just get it into the trunk.

Opened an XBean issue: https://issues.apache.org/jira/browse/XBEAN-122

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2
>            Reporter: Forrest Xia
>            Assignee: Jack Cai
>            Priority: Minor
>             Fix For: 2.1.4, 2.2
>
>         Attachments: Geronimo-4549_0307.patch
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Jack Cai (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679117#action_12679117 ] 

Jack Cai commented on GERONIMO-4549:
------------------------------------

Thanks Joe and David for your response!

Joe, if you look at the "Global context" after uninstalling the connector, you will see the connection factory name is still there.

I'll digg more into this.

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (GERONIMO-4549) JMS resource jndi entries are not removed after uninstalling the JMS connect adapter

Posted by "Joe Bohn (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/GERONIMO-4549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12678864#action_12678864 ] 

Joe Bohn commented on GERONIMO-4549:
------------------------------------

Perhaps I don't fully understand the problem ... but I attempted to recreate it and in my case the jndi entry was removed.  

I did the following:
- selected Services->JMS Resources from the navigation
- created a new resource group for ActiveMQ as follows:
    - specified a group of myRG and kept the defaults for all else
    - created a new connection factory under the Resource Group called myFactory and kept the defaults for all else
    - deployed the plan
- validated that the resource group was running and visible in the JMS Resources view, the J2EE Connectors view (and running) and the JNDI viewer under ResourceAdapterModule
- from the J2EE Connectors view I uninstalled my resource group rar
- It was removed from the J2EE Connectors view, the JMS Resources view, and the JNDI Viewer 

Am I completely missing the issue?

> JMS resource jndi entries are not removed after uninstalling the JMS connect adapter
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4549
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4549
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: ActiveMQ
>    Affects Versions: 2.1.4
>            Reporter: Forrest Xia
>            Priority: Blocker
>             Fix For: 2.1.4, 2.2
>
>
> Steps to reproduce this problem:
> 1. login admin console
> 2. Create a ActiveMQ resource connector with the wizard
> 3. Deploy it and check it is in running state
> 4. Click J2EE connector to uninstall it
> 5. Check JNDI viewer, you will see the JNDI entry still there, even you've uninstalled it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.