Web lists-archives.com

Re: Review Request 128664: Nested tags for Baloo

This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128664/

On September 4th, 2016, 9:51 a.m. UTC, Vishesh Handa wrote:

Please test out all possible workflows. Ideally it would be great if we could have automated tests for all the workflows with tags.

The only thing that could impact the existing workflow is a new hard length of 64. The new FT prefix is specific to the KIO Slave changes.

On September 4th, 2016, 9:51 a.m. UTC, Vishesh Handa wrote:

src/file/basicindexingjob.cpp (Diff revision 2)
            doc.addXattrBoolTerm(QByteArray("TAG-") + tag.toUtf8());
                doc.addXattrTerm(QByteArray("TA") + tag.toUtf8());

Alright. So now TA instead of being a prefix to store each word, it now stores the entire tag.

This breaks existing behaviour. That's typically frowned upon. Example - All tags with more than 1 word.

Perhaps it would just be easier if we would intrduce a new prefix.

Avoiding the term generator fixes false positives from = matches. These result in a file with tag Fire/Water showing up under "Fire" KIO Slave subdirectory and also in "Fire/Water" subdirectory. This behavior would be expected only from the : operator, and completely breaks the intent of the KIO Slave, to break a nested tag into a set of subdirectories that are then filled by inserting -only- exact tag matches from the path.

- James

On August 20th, 2016, 10:58 p.m. UTC, James Smith wrote:

Review request for Baloo and Vishesh Handa.
By James Smith.

Updated Aug. 20, 2016, 10:58 p.m.

Bugs: 334615
Repository: baloo


Index and query each tag as a single full term for generating recursed search results. Represent nested tags as recursed items in the Tags:// KIO Slave.


Compile, run


  • src/file/basicindexingjob.cpp (88bb59a01e5592abb74b1ab345bfc6765d35db57)
  • src/kioslaves/tags/kio_tags.cpp (de2e6d71945632e23a85f831878b4c431360731c)
  • src/lib/searchstore.cpp (060a4fd795ab858eb84526f93f827d09ee85db7c)

View Diff