You are viewing a plain text version of this content. The canonical link for it is here.

Posted to common-issues@hadoop.apache.org by "Daryn Sharp (Commented) (JIRA)" <ji...@apache.org> on 2012/01/09 18:14:39 UTC

[jira] [Commented] (HADOOP-7966) Add back in the older -getmerge addnl param and add a test case for the new -nl param.

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

Daryn Sharp commented on HADOOP-7966:
-------------------------------------

I'm not sure I agree with adding back the old behavior.  It used what should have been a path argument to decide if newlines should be added.  That behavior is inconsistent with all other commands.
                
> Add back in the older -getmerge addnl param and add a test case for the new -nl param.
> --------------------------------------------------------------------------------------
>
>                 Key: HADOOP-7966
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7966
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs
>    Affects Versions: 0.24.0
>            Reporter: Harsh J
>            Assignee: Harsh J
>
> This is to fix Eli's comments on the parent task:
> bq. This breaks a public API, why not preserve it but add in the improved one and update the docs to refer to it? Also, the test for this (HDFS-2070) should be done before the change.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira