[
https://issues.apache.org/jira/browse/HDFS-2768?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13187329#comment-13187329
]
Uma Maheswara Rao G commented on HDFS-2768:
-------------------------------------------
Hi Eli, Thanks for the review.
The classes like, ClinetProtocol, NamenodeProtocol..etc are not closeables. So,
we can't call directly IOUtils.cleanUp. Anyway we need to add the closeable
checks.
You mean, just need to remove, RPC.stopProxy invocation? and also no need to
re-factor DFSClient#closeConnectionToNamenode to DFSUtil? ( this is re-factored
because, i have seen similar piece of code in ConfiguredFailoverProxyProvider
also, So, in future we can use this api to close the connection)
> BackupNode stop can not close proxy connections because it is not a proxy
> instance.
> -----------------------------------------------------------------------------------
>
> Key: HDFS-2768
> URL: https://issues.apache.org/jira/browse/HDFS-2768
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: name-node
> Affects Versions: 0.24.0
> Reporter: Uma Maheswara Rao G
> Assignee: Uma Maheswara Rao G
> Attachments: HDFS-2768.patch, HDFS-2768.patch
>
>
> Observe this from BackupNode tests:
> java.lang.IllegalArgumentException: not a proxy instance
> at java.lang.reflect.Proxy.getInvocationHandler(Unknown Source)
> at org.apache.hadoop.ipc.RPC.stopProxy(RPC.java:557)
> at
> org.apache.hadoop.hdfs.server.namenode.BackupNode.stop(BackupNode.java:194)
> at
> org.apache.hadoop.hdfs.server.namenode.TestBackupNode.testCheckpoint(TestBackupNode.java:355)
> at
> org.apache.hadoop.hdfs.server.namenode.TestBackupNode.testBackupNode(TestBackupNode.java:241)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
> at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
> at java.lang.reflect.Method.invoke(Unknown Source)
> at junit.framework.TestCase.runTest(TestCase.java:168)
> at junit.framework.TestCase.runBare(TestCase.java:134)
> at junit.framework.TestResult$1.protect(TestResult.java:110)
--
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