Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#174 Store small views in SQLite #2427

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

lfcnassif
Copy link
Member

When finished, this should close #174.

TODO: Store views generated by MakePreviewTask, video thumbs and maybe HEIC views. Do you agree with the last two @wladimirleite?

@wladimirleite
Copy link
Member

TODO: Store views generated by MakePreviewTask, video thumbs and maybe HEIC views. Do you agree with the last two @wladimirleite?

Sure!

@wladimirleite
Copy link
Member

Just one minor detail, not sure if it is already the implemented behavior, when reading views (inside the analysis application), it could check if there is a file, beforing reading from the SQLite storage.
This would allow easy replacement (or creation) of view files (without having to deal with the SQLite).
Although it is not widely used (at least as far as I know), the possibility of externally creating views for certain items is a nice feature.

@lfcnassif
Copy link
Member Author

@fsicoli could you help testing this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Store small "view" files in sqlite containers
2 participants