[
https://issues.apache.org/jira/browse/FLUME-963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13206258#comment-13206258
]
[email protected] commented on FLUME-963:
-----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3869/
-----------------------------------------------------------
Review request for Flume.
Summary
-------
Adds verification to testAppend and testBadSimpleAppend.
Creates common verification method and changes all tests which output data to
this method.
Changes testBadSimpleAppend to all events are processed by sink.
This addresses bug FLUME-963.
https://issues.apache.org/jira/browse/FLUME-963
Diffs
-----
flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSink.java
798028f
Diff: https://reviews.apache.org/r/3869/diff
Testing
-------
Ran unit tests in this class in eclipse and all tests through maven.
Thanks,
Brock
> Add additional tes to TestHDFSEventSink and demystify existing tests
> --------------------------------------------------------------------
>
> Key: FLUME-963
> URL: https://issues.apache.org/jira/browse/FLUME-963
> Project: Flume
> Issue Type: Bug
> Components: Channel
> Affects Versions: v1.0.0
> Reporter: Brock Noland
> Fix For: v1.1.0
>
> Attachments: FLUME-963-test-verify.patch
>
>
> TestHDFSEventSink.testBadSimpleAppend only writes 57 events to the output as
> opposed to all 75. This can be verified by setting the environment variable
> hdfs_keepFiles or commenting out the tearDown method.
--
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
