D17313: Use Compact json formatting for storing document metadata

2018-12-02 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R293:b891716d336a: Use Compact json formatting for storing document metadata (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17313?vs=46716=46745

D17313: Use Compact json formatting for storing document metadata

2018-12-02 Thread Stefan Brüns
bruns added a comment. In D17313#370359 , @ngraham wrote: > Might this make the DB take up less space on disk? Yes, but not in a significant way - for me, the docData is less than 1% of the DB, this saves 10..50% of that. REPOSITORY

D17313: Use Compact json formatting for storing document metadata

2018-12-02 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. Might this make the DB take up less space on disk? REPOSITORY R293 Baloo BRANCH submit REVISION DETAIL https://phabricator.kde.org/D17313 To: bruns, #baloo, #frameworks, astippich, ngraham Cc: kde-frameworks-devel,

D17313: Use Compact json formatting for storing document metadata

2018-12-02 Thread Stefan Brüns
bruns added a comment. In D17313#370345 , @ngraham wrote: > This is backwards-compatible with the existing formal for retrieval, right? From the json standpoint, both are equivalent. REPOSITORY R293 Baloo BRANCH submit REVISION

D17313: Use Compact json formatting for storing document metadata

2018-12-02 Thread Nathaniel Graham
ngraham added a comment. This is backwards-compatible with the existing formal for retrieval, right? REPOSITORY R293 Baloo BRANCH submit REVISION DETAIL https://phabricator.kde.org/D17313 To: bruns, #baloo, #frameworks, astippich, ngraham Cc: kde-frameworks-devel, ashaposhnikov,

D17313: Use Compact json formatting for storing document metadata

2018-12-02 Thread Alexander Stippich
astippich accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH submit REVISION DETAIL https://phabricator.kde.org/D17313 To: bruns, #baloo, #frameworks, astippich, ngraham Cc: kde-frameworks-devel, ashaposhnikov, michaelh, astippich,

D17313: Use Compact json formatting for storing document metadata

2018-12-02 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, Frameworks, astippich, ngraham. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY Document metadata is stored json encoded in the