[
https://issues.apache.org/jira/browse/HADOOP-4226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12635415#action_12635415
]
Yuri Pradkin commented on HADOOP-4226:
--------------------------------------
bq. old patches are not deleted, as they orphan the discussion about them.
Sorry about that: started deleting inconsequential patches and got carried away.
bq. If this could include a short summary of the strategy employed and a few
more comments, it would be more readily understandable the next time a change
is proposed.
I'm not a java programmer, so I could use some advice here on where would such
summary go: javadoc comments (/**...**/) or the policy is to keep them short
and the longer comments should go somewhere else?
bq. CR and LF should be final, as in the previous patch
If people were to re-use this class with different newline separators, they
might find it useful to be able to change CR and LF. Maybe they should not be
static and be public, so they can simply be assigned to?
> LineReader::readLine cleanup
> ----------------------------
>
> Key: HADOOP-4226
> URL: https://issues.apache.org/jira/browse/HADOOP-4226
> Project: Hadoop Core
> Issue Type: Improvement
> Components: mapred
> Affects Versions: 0.19.0
> Reporter: Yuri Pradkin
> Assignee: Yuri Pradkin
> Priority: Minor
> Attachments: HADOOP-4226.patch
>
>
> I've been looking at HADOOP-4010 and realized that readLine is pretty
> convoluted. I changed the implementation which made it hopefully a little
> easier to read/validate/understand.
> I've had some problems testing it locally, so I'll submit it for Hudson to
> test.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
