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

Nandakumar commented on HDFS-12050:
-----------------------------------

Thanks [~vagarychen] for the review.
We only have {{RemovalListener}} configured for {{clientCache}}, so I think 
it's better to call {{clientCache.cleanup()}} after 
{{clientCache.invalidateAll()}}. It will make sure that RemovalListener is 
invoked immediately.
I will upload a new patch with the same.

> Ozone: StorageHandler: Implementation of "close" for releasing resources 
> during shutdown
> ----------------------------------------------------------------------------------------
>
>                 Key: HDFS-12050
>                 URL: https://issues.apache.org/jira/browse/HDFS-12050
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>            Reporter: Nandakumar
>            Assignee: Nandakumar
>            Priority: Minor
>         Attachments: HDFS-12050-HDFS-7240.000.patch
>
>
> When we use DistributedStorageHandler and call {{newKeyWriter}}, it will 
> create {{XceiverClientSpi}} and add it to clientCache which will create a 
> non-daemon {{nioEventLoopGroup}} thread. Since 
> {{XceiverClientManager#releaseClient}} doesn't invalidate the client, close 
> is not called on the client object. Cleanup is triggered as part of both 
> eviction as well as releaseClient, but the connection is closed only when 
> both of the conditions are satisfied.
> This {{StorageHandler#close}} can be used to close the connections during 
> shutdown.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to