[jira] [Commented] (DIRSERVER-2190) there is thread leak when did following operations: ADD,DELETE,MODIFY,MOVE,RENAME

["paul qing (JIRA)" <ji...@apache.org>]

    [ 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)