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

Allen Wittenauer commented on HADOOP-13079:
-------------------------------------------

I've removed the supportability flag. This is not a supportability issue given 
that making -q the default would harm the ability to use ls as a canary prior 
to looking at other content.

Let me be explicit on my stance:

I'm +1 on adding a -q option.

I'm -1 on making -q the default given that:

a) It's not standardized behavior amongst all of the platforms that Apache 
Hadoop runs

b) It's not expected behavior relative to the rest of Apache Hadoop

c) It's not feasible to actually make it expected behavior compared to the rest 
of Apache Hadoop given the proliferation of places where raw file and directory 
names are printed to the console


> Add -q to fs -ls to print non-printable characters
> --------------------------------------------------
>
>                 Key: HADOOP-13079
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13079
>             Project: Hadoop Common
>          Issue Type: Bug
>    Affects Versions: 2.6.0
>            Reporter: John Zhuge
>            Assignee: John Zhuge
>
> Add option {{-q}} to "hdfs dfs -ls" to print non-printable characters as "?". 
> Non-printable characters are defined by 
> [isprint(3)|http://linux.die.net/man/3/isprint] according to the current 
> locale.
> Default to {{-q}} behavior on terminal; otherwise, print raw characters. See 
> the difference in these 2 command lines:
> * {{hadoop fs -ls /dir}}
> * {{hadoop fs -ls /dir | od -c}}
> In C, {{isatty(STDOUT_FILENO)}} is used to find out whether the output is a 
> terminal. Since Java doesn't have {{isatty}}, I will use JNI to call C 
> {{isatty()}} because the closest test {{System.console() == null}} does not 
> work in some cases.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to