Uploaded image for project: 'Directory ApacheDS'
  1. Directory ApacheDS
  2. DIRSERVER-2190

there is thread leak when did following operations: ADD,DELETE,MODIFY,MOVE,RENAME

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-M20, 2.0.0-M21, 2.0.0-M22, 2.0.0-M23
    • Fix Version/s: 2.0.0-M24
    • Component/s: None
    • Labels:
      None
    • Environment:
      windows 7 ,linux.

      Description

      the org.apache.directory.server.core.event.EventInterceptor class didn't override the destory method. and the thread pool didn't call shutdown method.i think it should call thread pool's shutdown method in the inteceptor's destory method and it should call destory inteceptor in the org.apache.directory.server.core.DefaultDirectoryService's shutdown method. such as:
      public synchronized void shutdown() throws Exception
      {**********
      **************
      this.destoryInteceptors();
      }

      private void destoryInteceptors() {
      for (Interceptor interceptor : interceptors)

      { interceptor.destroy(); }

      }

        Activity

        Hide
        paulqing paul qing added a comment -

        the EventInterceptor class should override method as following:
        @Override
        public void destroy(){
        if (executor != null)

        { executor.shutdown(); }

        }

        Show
        paulqing paul qing added a comment - the EventInterceptor class should override method as following: @Override public void destroy(){ if (executor != null) { executor.shutdown(); } }
        Hide
        elecharny Emmanuel Lecharny added a comment -
        Show
        elecharny Emmanuel Lecharny added a comment - Thanks a lot for the patch. It has been applied in http://svn.apache.org/viewvc?rev=1792186&view=rev and http://svn.apache.org/viewvc?rev=1792184&view=rev

          People

          • Assignee:
            Unassigned
            Reporter:
            paulqing paul qing
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development