Web lists-archives.com

Re: Review Request 129703: [baloo_file_extractor] Limit CPU usage

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

On December 26th, 2016, 1:24 a.m. CET, Aleix Pol Gonzalez wrote:

src/file/extractor/app.cpp (Diff revision 1)
        QString url = QFile::decodeName(m_tr->documentUrl(id));
        QString url = QFile::decodeName(m_tr->documentUrl(id));

FWIW, something looks incorrect here, as it seems be using Urls as paths. I know it's unrelated but maybe you can check?

On December 26th, 2016, 5:48 a.m. CET, Anthony Fieroni wrote:

QByteArray Transaction::documentUrl(quint64 id) const Url is stored as QByteArray in db, it looks correct.

It's fine that it's a QByteArray, the problem is whether it has the "file://" prefix.

- Aleix

On December 25th, 2016, 2:27 p.m. CET, Anthony Fieroni wrote:

Review request for Baloo and Vishesh Handa.
By Anthony Fieroni.

Updated Dec. 25, 2016, 2:27 p.m.

Repository: baloo


Processing large directories, +5000 files, can be CPU eater. Large file, itself, can be another issue.


  • src/file/extractor/app.cpp (97332469)

View Diff