Code review comment for lp:~jtatum/ubuntu-accomplishments-daemon/lp-caching-part-1

Revision history for this message
Rafał Cieślak (rafalcieslak256) wrote :

This is a valid point, and I have to agree that particular collections shouldn't work on the same cache file. I guess the most elegant solution then would be to simply use separate files for each collection. I don't really think we need another project for that, this really could be done on scripts' side. The scripts are responsible for fetching the data, so ideally they would care about their cache too. I imagine such caching mechanism might be even merged into a single file (or a few) that would be shipped alongside our current scripts and included/imported by them, so that it would be easily re-usable in other collections.

As a side note, since we now have a lot of AU accomplishments, they would make a great use of their own cache, for they all download the very same file.

« Back to merge proposal