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

Ayush Saxena commented on HDFS-15766:
-------------------------------------

Seems there is a change in {{TestRouterRpc.}}

{code:java}
         verifyFileExists(routerFS, snapshotFolder + "/.snapshot/newsnap"));
 
     // Cleanup
-    routerProtocol.delete(mountPoint, true);
+    nnProtocol.delete(mountPoint, true);
   }
{code}

Why do we need that, we did a mkdir using routerProtocol, the delete should 
also work on that?

> RBF: MockResolver.getMountPoints() breaks the semantic of 
> FileSubclusterResolver.
> ---------------------------------------------------------------------------------
>
>                 Key: HDFS-15766
>                 URL: https://issues.apache.org/jira/browse/HDFS-15766
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Jinglun
>            Assignee: Jinglun
>            Priority: Major
>         Attachments: HDFS-15766.001.patch
>
>
> MockResolver.getMountPoints() breaks the semantic of 
> FileSubclusterResolver.getMountPoints(). Currently it returns null when the 
> path is a mount point and no mount points are under the path. 
> {quote}Return zero-length list if the path is a mount point but there are no 
> mount points under the path.
> {quote}
>  
> This is required by router federation rename. I found this bug when writing 
> unit test for the rbf rename. Let's fix it here to avoid mixing up with the 
> router federation rename.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to