Changing files names so that memory mapped file is not same.

Changing files names so that memory mapped file is not same.

Roshan,

This patch seems to be ok for the tests to pass, but it does look like it will leave all of the original files lurking around in the Windows case(since we use deleteQuietly in tearDown which will eat the exceptions). Can that be fixed somehow?

I have seen that issue of some files being not deleted. AFAICT It is not specific to this test/patch.
Here i have setup FileUtils.forceDeleteOnExit() inside the test as far as possible.

I think the proper cleanup on windows needs to be addressed in another jira. agree ?

Roshan,

I missed one thing yesterday. If you look at the test you can see that the channel is started using the same checkpoint directory later (after checkpoint rebuilder is done rebuilding). This issue with doing that is that the channel will restart with "checkpointDir/checkpoint" as the checkpoint and the new file created by checkpoint rebuilder is ignored. We should probably create the new checkpoint in a different directory and start the file channel with that directory as the checkpoint directory to work around this.

As far as doing the windows specific clean up in a follow up - that sounds ok, but I'd like to make sure that if new directories (as I suggested above) are added to the tests, they get cleaned up when the test is done.

Filed FLUME-2149

Incorporating review feedback from Hari to change checkpoint dir instead of checkpoint file