[Asterisk-code-review] media cache: Add a core API and facade for a backend agnosti... (asterisk[master])

[Matt Jordan]

Matt Jordan has posted comments on this change.

Change subject: media cache: Add a core API and facade for a backend agnostic media cache
......................................................................


Patch Set 2:

> (1 comment)

I think you have described what is necessary for this to work, assuming we want to use the bucket API for any of this. To an extent, it feels like I'm fighting against sorcery, which bucket is built upon. That's unfortunate, as the bucket API should be perfect for this problem:
* I need to track local files
* I need to track metadata associated with them
* The implementation of where those files come from could change (or it would be nice to support multiple implementations)

I think the only path forward - or at least one where I don't abuse sorcery - is either for sorcery to be extended with a callback in its API that supports what this needs, or for the use of bucket to be abandoned.

-- 
To view, visit https://gerrit.asterisk.org/487
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I11227abbf14d8929eeb140ddd101dd5c3820391e
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: No