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

Yiqun Lin commented on HDFS-14209:
----------------------------------

[~shubham.dewan], in the {{RouterQuotaManager#getPaths(path);}}, it can include 
the itself path. Is still following change needed?
{code}
+    if (locations.size() >= 1) {
+      return locations;
+    } else {
+      locations.addAll(rpcServer.getLocationsForPath(path, true, false));
+      return locations;
+    }
{code}

For the test, I think we maybe call the {{updateService.periodicInvoke();}} 
firstly (for loading path to QuotaManager), then do the setQuota call. Current 
order of these two lines seem not right.

> RBF: setQuota() through router is working for only the mount Points under the 
> “Source” column in MountTable
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: HDFS-14209
>                 URL: https://issues.apache.org/jira/browse/HDFS-14209
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Shubham Dewan
>            Assignee: Shubham Dewan
>            Priority: Major
>         Attachments: HDFS-14209-HDFS-13891.002.patch, 
> HDFS-14209-HDFS-13891.003.patch, HDFS-14209.001.patch
>
>
> Through router we are only able to setQuota for the directories under the 
> Source column of the mount table.
>  For any other directories apart from mount table entry ==> No remote 
> locations available IOException is thrown.
>  Should be able to setQuota for all the directories if present.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to