[ 
https://issues.apache.org/jira/browse/DIRSERVER-2190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15978382#comment-15978382
 ] 

paul qing commented on DIRSERVER-2190:
--------------------------------------

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

> there is thread leak when did following operations: 
> ADD,DELETE,MODIFY,MOVE,RENAME
> ---------------------------------------------------------------------------------
>
>                 Key: DIRSERVER-2190
>                 URL: https://issues.apache.org/jira/browse/DIRSERVER-2190
>             Project: Directory ApacheDS
>          Issue Type: Bug
>    Affects Versions: 2.0.0-M20, 2.0.0-M21, 2.0.0-M22, 2.0.0-M23
>         Environment: windows 7 ,linux.
>            Reporter: paul qing
>
> 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();
>               }
>       }



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to