[
https://issues.apache.org/jira/browse/MAPREDUCE-5587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13799489#comment-13799489
]
Jonathan Eagles commented on MAPREDUCE-5587:
--------------------------------------------
Thanks for the review, Ravi.
bq. in testCompress(), you didn't have to remove separator. I'd rather check
\u0001 than \t.
It's true that removing the separator wasn't technically part of this JIRA, but
I chose to take it out since the custom separator test was already testing that.
bq. Do you think we should clean up the output files in an @After ? Or do you
think its more useful to have them around after the test for debugging? I could
go either way.
Cleaned up temp files from this test
> TestTextOutputFormat fails on JDK7
> ----------------------------------
>
> Key: MAPREDUCE-5587
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-5587
> Project: Hadoop Map/Reduce
> Issue Type: Bug
> Affects Versions: 3.0.0, 2.3.0, 0.23.10
> Reporter: Jonathan Eagles
> Assignee: Jonathan Eagles
> Attachments: MAPREDUCE-5587.patch, MAPREDUCE-5587.patch
>
>
> the test method run order on JDK7 is not fixed causing test method
> inter-dependencies to show themselves.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
