Github user pzygielo commented on a diff in the pull request:

    https://github.com/apache/tomee/pull/262#discussion_r241051038
  
    --- Diff: 
container/openejb-loader/src/main/java/org/apache/openejb/observer/ObserverManager.java
 ---
    @@ -74,6 +80,12 @@ public boolean addObserver(final Object observer) {
             }
         }
     
    +    /**
    +     * Remove the observable object so this object doesn't have any 
observers
    --- End diff --
    
    I think only specified `observer` will be removed, and all the rest will be 
kept.
    And observable will not be removed.


---

Reply via email to