You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@flume.apache.org by "Johny Rufus (JIRA)" <ji...@apache.org> on 2015/01/05 20:09:34 UTC

[jira] [Comment Edited] (FLUME-2568) Additional fix for TestReliableSpoolingFileEventReader

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

Johny Rufus edited comment on FLUME-2568 at 1/5/15 7:09 PM:
------------------------------------------------------------

Attaching a patch, that takes care of the failing test 


was (Author: jrufus):
Attaching a patch, that takes of the failing test 

> Additional fix for TestReliableSpoolingFileEventReader
> ------------------------------------------------------
>
>                 Key: FLUME-2568
>                 URL: https://issues.apache.org/jira/browse/FLUME-2568
>             Project: Flume
>          Issue Type: Bug
>    Affects Versions: v1.5.1
>            Reporter: Johny Rufus
>            Assignee: Johny Rufus
>             Fix For: v1.6.0
>
>         Attachments: FLUME-2568.patch
>
>
> TestReliableSpoolingFileEventReader needs some more modification in addition to the fix provided in FLUME-2505
> 1) Needed to add semaphore2 to ensure that new file gets created before the second listing, without this readEventsForFilesInDir() can complete before the new file gets created
> 2) had to remove renameTo and instead create the newfile, for the directory listing to pick the new file 



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