[
https://issues.apache.org/jira/browse/MAPREDUCE-7263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17034454#comment-17034454
]
Hudáky Márton Gyula commented on MAPREDUCE-7263:
------------------------------------------------
The modification consisted of removing the validateTargetPath() function and
place where it is called. It was a test itself which become obsolete by the
development of Hadoop, so there is no need to unit test this change.
> 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: [email protected]
For additional commands, e-mail: [email protected]
