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

Tsz Wo (Nicholas), SZE commented on MAPREDUCE-1579:
---------------------------------------------------

> Nicholas, the patch looks good and you did some great refactoring also. ...
Thanks, Rodrigo for taking a look.  For the refactoring, you probably mean the 
change of the map input vale class from Text to HarEntry (and removing 
MapStat).  It is necessary because the original codes uses Text to store paths 
in a SequenceFile, which again has the space problem.

> archive: check and possibly replace the space charater in paths
> ---------------------------------------------------------------
>
>                 Key: MAPREDUCE-1579
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1579
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>          Components: harchive
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Mahadev konar
>         Attachments: m1579_20100310.patch
>
>
> Since the space character is used as a separator in the index files, it won't 
> work if there are spaces in the path (see also HADOOP-6591).  The archive 
> tools should 
> # detect if there are spaces in the paths and 
> # provide an option to replace it with some other characters.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to