[ https://issues.apache.org/jira/browse/HADOOP-8728?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17616850#comment-17616850 ]
ASF GitHub Bot commented on HADOOP-8728: ---------------------------------------- ashutoshcipher commented on code in PR #5010: URL: https://github.com/apache/hadoop/pull/5010#discussion_r994280180 ########## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/Display.java: ########## @@ -217,8 +216,8 @@ protected void processPath(PathData item) throws IOException { protected class TextRecordInputStream extends InputStream { SequenceFile.Reader r; - Writable key; - Writable val; + Object key; Review Comment: Hi @goiri. Thanks for your review. Currently - The Display command (fs -text) currently reads only Writable-based SequenceFiles. This isn't necessary to do, and prevents reading non-Writable-based serialization in SequenceFiles from the shell. > Display (fs -text) shouldn't hard-depend on Writable serialized sequence > files. > ------------------------------------------------------------------------------- > > Key: HADOOP-8728 > URL: https://issues.apache.org/jira/browse/HADOOP-8728 > Project: Hadoop Common > Issue Type: Bug > Components: fs > Affects Versions: 2.6.0 > Reporter: Harsh J > Assignee: Ashutosh Gupta > Priority: Minor > Labels: BB2015-05-TBR, pull-request-available > Attachments: HADOOP-8728-002.patch, HADOOP-8728-003.patch, > HADOOP-8728.patch > > > The Display command (fs -text) currently reads only Writable-based > SequenceFiles. This isn't necessary to do, and prevents reading > non-Writable-based serialization in SequenceFiles from the shell. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org