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

Suresh Srinivas commented on HADOOP-8218:
-----------------------------------------

Todd, I agree with Tom. The proxy must implement Closeable or have associated 
invoker with Closeable, is the new requirement. Without this, we were silently 
ignoring the failure to stop the proxy. The mocks also need to satisfy this 
requirement.

Thanks for doing a patch for option #3. +1 for the patch.
                
> RPC.closeProxy shouldn't throw error when closing a mock
> --------------------------------------------------------
>
>                 Key: HADOOP-8218
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8218
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: ipc, test
>    Affects Versions: 0.24.0
>            Reporter: Todd Lipcon
>            Assignee: Todd Lipcon
>            Priority: Critical
>         Attachments: hadoop-8218.txt, hadoop-8218.txt
>
>
> HADOOP-8202 changed the behavior of RPC.stopProxy() to throw an exception if 
> called on an object which doesn't implement Closeable. Unfortunately, we use 
> mock objects in many test cases, and those mocks don't implement Closeable. 
> This is causing TestZKFailoverController to fail in trunk, for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to