Add CachingStorage
AbandonedPublic
Actions

Authored by ervin on Oct 9 2015, 9:22 AM.

Details

Reviewers

bensi

Summary

The current implementation only forwards the call to the Storage.

Repository

R4 Zanshin

Lint

Lint Skipped

Unit

Unit Tests Skipped

bensi retitled this revision from to Add CachingStorage.

bensi updated this object.

bensi edited the test plan for this revision. (Show Details)

bensi added a reviewer: ervin.

bensi set the repository for this revision to R4 Zanshin.

bensi added a project: Zanshin.

ervin requested changes to this revision.Oct 9 2015, 11:35 AM

ervin edited edge metadata.

ervin added inline comments.

src/akonadi/akonadicachingstorage.cpp
32	Probably needs to be controllable on the ctor. See my other comment on the why. ;-)
src/app/dependencies.cpp
58 ↗	(On Diff #954)	You also need to make sure the acceptance tests use the CachingStorage on top of the storage implementation used there. It should be in this commit IMO to catch regressions early and not forget to adjust the acceptance tests later.