You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-issues@hadoop.apache.org by "Robert Kanter (JIRA)" <ji...@apache.org> on 2018/01/18 20:42:00 UTC
[jira] [Comment Edited] (MAPREDUCE-6995) Uploader tool for
Distributed Cache Deploy documentation
[ https://issues.apache.org/jira/browse/MAPREDUCE-6995?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16331177#comment-16331177 ]
Robert Kanter edited comment on MAPREDUCE-6995 at 1/18/18 8:41 PM:
-------------------------------------------------------------------
Some minor comments:
# "The tool then returns a suggestion how to set ..." should be "The tool then returns a suggestion *of* how to set ..."
# "Defaults to the default filesystem set by fs.defaultFS." - fs.defaultFs should have the ` to make it monospaced.
# Can you double check that paths starting with just {{hdfs:/}} work? (as opposed to {{hdfs://}})
# The explanations of the arguments (e.g. {{-initialReplication}}) should have the ` around the argument names to make them monospaced.
# "If this value is set to low like a constant 10 ..." is more clear and consistent with the next sentence if it's written as "If this is set to a low value like 10 ..."
# The description for {{-acceptableReplication}} should say what it actually does, not just the requirements for it. Something like "The tool will wait until the tarball has been replicated this number of times before exiting."
# "The frameworkuploader tool has the following arguments to control, which jars end up in the framework tarball:" should not have the comma.
# "This is an input classpath that is iterated through. jars files found will be added to the tarball. Defaults to the classpath." should be "This is *the* input classpath to source jar files from to add to the tarball. *It defaults to the classpath as returned by the {{hadoop classpath}} command.*"
# "This is a comma separated regex array to filter the jar file names to include from the class path." should be "This is a comma separated regex array to filter the jar file names to *exclude* from the class path."
# It might be good to give an example of {{-nosymlink}} as it's not clear just from the description of what this does. Something like "For example, {{/a/foo.jar}} and a symlink {{/a/bar.jar}} that points to {{/a/foo.jar}} would normally add foo.jar and bar.jar to the tarball as separate files despite them actually being the same file. This flag would make the tool exclude {{/a/bar.jar}} so only one copy of the file is added."
# In the {{testExplicitFilesystem}} test, we should have a similar test where {{-target}} and {{FS_DEFAULT_NAME_KEY}} have different filesystems.
# The {{frameworkupload}} command should be added to the MapredCommands.md file
was (Author: rkanter):
Some minor comments:
# "The tool then returns a suggestion how to set ..." should be "The tool then returns a suggestion *of* how to set ..."
# "Defaults to the default filesystem set by fs.defaultFS." - fs.defaultFs should have the ` to make it monospaced.
# Can you double check that paths starting with just {{hdfs:/}} work? (as opposed to {{hdfs://}})
# The explanations of the arguments (e.g. {{-initialReplication}}) should have the ` around the argument names to make them monospaced.
# "If this value is set to low like a constant 10 ..." is more clear and consistent with the next sentence if it's written as "If this is set to a low value like 10 ..."
# The description for {{-acceptableReplication}} should say what it actually does, not just the requirements for it. Something like "The tool will wait until the tarball has been replicated this number of times before exiting."
# "The frameworkuploader tool has the following arguments to control, which jars end up in the framework tarball:" should not have the comma.
# "This is an input classpath that is iterated through. jars files found will be added to the tarball. Defaults to the classpath." should be "This is *the* input classpath to source jar files from to add to the tarball. *It defaults to the classpath as returned by the {{hadoop classpath}} command.*"
# "This is a comma separated regex array to filter the jar file names to include from the class path." should be "This is a comma separated regex array to filter the jar file names to *exclude* from the class path."
# It might be good to give an example of {{-nosymlink}} as it's not clear just from the description of what this does. Something like "For example, {{/a/foo.jar}} and a symlink {{/a/bar.jar}} that points to {{/a/foo.jar}} would normally add foo.jar and bar.jar to the tarball as separate files despite them actually being the same file. This flag would make the tool exclude {{/a/bar.jar}} so only one copy of the file is added."
# In the {{testExplicitFilesystem}} test, we should have a similar test where {{-target}} and {{FS_DEFAULT_NAME_KEY}} have different filesystems.
> Uploader tool for Distributed Cache Deploy documentation
> --------------------------------------------------------
>
> Key: MAPREDUCE-6995
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6995
> Project: Hadoop Map/Reduce
> Issue Type: Sub-task
> Reporter: Miklos Szegedi
> Assignee: Miklos Szegedi
> Priority: Major
> Attachments: MAPREDUCE-6995.000.patch, MAPREDUCE-6995.001.patch, MAPREDUCE-6995.002.patch, MAPREDUCE-6995.003.patch
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-help@hadoop.apache.org