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

Adam Antal commented on MAPREDUCE-7263:
---------------------------------------

Thanks for uploading the patch [~mhudaky]!

+1 (non-binding).

> Remove obsolete validateTargetPath() from FrameworkUploader
> -----------------------------------------------------------
>
>                 Key: MAPREDUCE-7263
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-7263
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>          Components: mrv2
>    Affects Versions: 3.2.1
>            Reporter: Adam Antal
>            Assignee: Hudáky Márton Gyula
>            Priority: Major
>         Attachments: MAPREDUCE-7263.001.patch
>
>
> The following piece of code has been in the FrameworkUploader class for a 
> while:
> {code:java}
>   private void validateTargetPath() throws UploaderException {
>     if (!target.startsWith("hdfs:/") &&
>         !target.startsWith("file:/")) {
>       throw new UploaderException("Target path is not hdfs or local " + 
> target);
>     }
>   }
> {code}
> As parts of MR and YARN has evolved, you can run YARN with OzoneFS as storage 
> for instance - so HDFS (or local) is not a requirement anymore. Also it does 
> not make sense to add other suitable filesystems when they're tested against 
> the MR framework, so I'm suggesting to remove the check.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

Reply via email to