Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/ --- (Updated Aug. 20, 2015, 5:10 p.m.) Status -- This change has been ma

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/#review84070 --- Ship it! Looks great! Thanks for debugging this and providing

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/ --- (Updated Aug. 18, 2015, 7:32 p.m.) Review request for Baloo, Bhushan Shah

Re: Review Request 124794: Create one Database instance per process

> On Aug. 18, 2015, 6:46 p.m., Vishesh Handa wrote: > > src/tools/balooctl/main.cpp, line 127 > > > > > > We just put the extra '/' over here. ah, my stupidity! - Ashish -

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/#review84014 --- src/tools/balooctl/main.cpp (line 125)

Re: Review Request 124794: Create one Database instance per process

> On Aug. 18, 2015, 6:26 p.m., Vishesh Handa wrote: > > src/tools/balooctl/main.cpp, line 125 > > > > > > I think you can use the `fileIndexDbPath` function over here? Actually it ends with one extra "/" as it's

Re: Review Request 124794: Create one Database instance per process

> On Aug. 17, 2015, 7:50 p.m., Vishesh Handa wrote: > > autotests/unit/lib/filefetchjobtest.cpp, line 92 > > > > > > What is this used for? > > Ashish Bansal wrote: > I have removed autotests/unit/lib/db.cpp

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/#review84011 --- autotests/unit/lib/CMakeLists.txt (line 24)

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/ --- (Updated Aug. 18, 2015, 6:08 p.m.) Review request for Baloo, Bhushan Shah

Re: Review Request 124794: Create one Database instance per process

> On Aug. 17, 2015, 7:50 p.m., Vishesh Handa wrote: > > autotests/unit/lib/filefetchjobtest.cpp, line 92 > > > > > > What is this used for? I have removed autotests/unit/lib/db.cpp which contained these functions

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/ --- (Updated Aug. 18, 2015, 12:59 p.m.) Review request for Baloo, Bhushan Sha

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/#review83960 --- Looks good. I didn't realize making the database global would

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/ --- (Updated Aug. 17, 2015, 6:32 p.m.) Review request for Baloo, Bhushan Shah

Re: Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/#review83956 --- src/engine/database.h (line 47)

Review Request 124794: Create one Database instance per process

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124794/ --- Review request for Baloo, Bhushan Shah and Vishesh Handa. Bugs: 350247