Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Flume, mail # dev - Re: Review Request 11536: Fix for TestFlumeEventQueue failures in FileChannel on Windows


Copy link to this message
-
Re: Review Request 11536: Fix for TestFlumeEventQueue failures in FileChannel on Windows
Alexander Alten-Lorenz 2013-07-17, 17:00

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11536/#review23294
-----------------------------------------------------------
How's about using the flag FILE_SHARE_DELETE - but will be only supported in J7, unfortunately (http://www.docjar.com/html/api/sun/nio/fs/WindowsFileAttributeViews.java.html)?

- Alexander Alten-Lorenz
On May 30, 2013, 4:37 a.m., Roshan Naik wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11536/
> -----------------------------------------------------------
>
> (Updated May 30, 2013, 4:37 a.m.)
>
>
> Review request for Flume.
>
>
> Bugs: FLUME-2058
>     https://issues.apache.org/jira/browse/FLUME-2058
>
>
> Repository: flume-git
>
>
> Description
> -------
>
> Windows does not allow deletion of files that are in use. Java does not yet allow explicitly unmapping memory mapped files so until these files are unmapped by the GC, we cannot delete them explicitly.
>
> Checkpoint files are being reused across tests. Deletion of checkpoint files fails in these tests as the file memory mapped.
>
> Fix: Not reusing checkpoint files to avoid deletion problems. Also ensuring files are setup to be deleted by jvm on exit.
>
>
> Diffs
> -----
>
>   flume-ng-channels/flume-file-channel/src/test/java/org/apache/flume/channel/file/TestFlumeEventQueue.java 203cbf2
>
> Diff: https://reviews.apache.org/r/11536/diff/
>
>
> Testing
> -------
>
> TestFlumeEventQueue test now pass on Windows.
>
>
> Thanks,
>
> Roshan Naik
>
>