[GitHub] [hadoop] xinglin commented on pull request #4128: Hadoop-18169. getDelegationTokens in ViewFs should also fetch the token from fallback FS

[GitBox <gi...@apache.org>]

xinglin commented on PR #4128:
URL: https://github.com/apache/hadoop/pull/4128#issuecomment-1099464773

   > This change alone seems fine. It would be good and helpful for others reviewing to identify the differences in viewfs to explain why the rest of the tests couldn't be backported, but thats more of a nice to have, not required to ship it
   
   @mccormickt12 thanks for taking a quick review. 
   
   create() method in InternalDirOfViewFs diverged between trunk and branch-2.10. In trunk, when fallbackFS is configured, we can create a file but in branch-2.10, it does not do the check whether fallbackFS exists and simply throws "read-only fs" exception, thus failing these unit tests. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org