Skip to content
This repository has been archived by the owner on Aug 25, 2020. It is now read-only.

Use VanillaChronicle for EventStore log rotation #348

Open
TobiasMende opened this issue Apr 7, 2014 · 5 comments
Open

Use VanillaChronicle for EventStore log rotation #348

TobiasMende opened this issue Apr 7, 2014 · 5 comments
Assignees
Labels
Milestone

Comments

@TobiasMende
Copy link
Contributor

... when available

@danbim
Copy link
Member

danbim commented Apr 7, 2014

Could you please provide a link to the issue in the Chronicle issue tracker? Or is there no such thing?

@TobiasMende
Copy link
Contributor Author

No there isn't an issue for that. The only issue related to the RollingChronicle is this one:
https://github.com/OpenHFT/Java-Chronicle/issues/33

@danbim danbim modified the milestones: 0.9, 0.9.1 May 26, 2014
@TobiasMende
Copy link
Contributor Author

Just found the "VanillaChronicle" which is part of the chronicle API version 3.2.1 and offers file rotation (https://github.com/OpenHFT/Java-Chronicle/blob/master/docs/VanillaChronicle.md)

It seems not too difficult to replace the RollingChronicle in the event store implementation hence the interfaces are the same or at least very similar.

@TobiasMende TobiasMende self-assigned this Aug 6, 2014
@TobiasMende
Copy link
Contributor Author

Almost finished but currently blocked by https://github.com/OpenHFT/Java-Chronicle/pull/78

The test cases for the cycling event store fail due to a 'corrupted length' error (IllegalStataException) which is addressed in the pull request above.

@danbim danbim modified the milestones: 0.9.1, 0.10 Oct 15, 2014
@TobiasMende TobiasMende changed the title Switch EventStore to RollingChronicle Use VanillaChronicle for EventStore log rotation Dec 29, 2014
@TobiasMende
Copy link
Contributor Author

Chronicle 3.3.3 now supports log rotation for VanillaChronicle. The new ChronicleQueueBuilder offers configuration parameters for cycleFormat and cycleLength which affect the naming and size the of chronicle files.

The authors recommend to use VanillaChronicle instead of RollingChronicle from now on. I'll change the implementation according to their recommendation.

@danbim danbim modified the milestones: 0.10, 0.11 Jun 1, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants