search

itm / testbed-runtime

WISEBED Wireless Sensor Network Testbed Infrastructure Software
https://github.com/itm/testbed-runtime/wiki
15 stars 11 forks source link

Use VanillaChronicle for EventStore log rotation #348

Open TobiasMende opened 10 years ago

TobiasMende commented 10 years ago

... when available

danbim commented 10 years ago

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

TobiasMende commented 10 years ago

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

TobiasMende commented 10 years ago

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 commented 10 years ago

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.

TobiasMende commented 9 years ago

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.