[ 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