Removed the separate thread for resource score cache processing
ClosedPublic
Actions

Authored by ivan on Apr 28 2018, 11:08 AM.

Details

Reviewers

davidedmundson

Commits

Repository

R161 KActivity Manager Service

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

ivan requested review of this revision.Apr 28 2018, 11:08 AM

Could that also explain the QtSql crashes I'm seeing daily in bookmarks runner?

If KRunner has multiple threads accessing the same sqlite database, then yes.

(if the question was whether this patch will fix krunner - then no - this has nothing to do with krunner)

This revision is now accepted and ready to land.Apr 28 2018, 12:48 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			src/service/plugins/sqlite/ResourceScoreMaintainer.h (4 lines)
M			src/service/plugins/sqlite/ResourceScoreMaintainer.cpp (64 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	33243	5451dba		Apr 28 2018, 11:08 AM	★	★
Diff 2	33244	5451dba	R161:74512fe9a50b2ca2ec6756b08f85944487c44ddc	Apr 28 2018, 12:49 PM	★	★