D24540: [Database] Disable read-ahead to reduce thrashing

2019-10-11 Thread Kai Krakow 
hurikhan77 added a comment.


  Here's a quote:
  https://github.com/bmatsuo/lmdb-go/issues/118#issuecomment-325449496
  
  I suspect that we can safely say: The Baloo DB is always bigger than the 
available RAM (we shouldn't fight with what the desktop needs, so let's say we 
have maybe 20% of the installed RAM at our disposal which is already a high 
assumptions for typical systems).

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D24540

To: hurikhan77, ngraham, #baloo, bruns
Cc: bruns, kde-frameworks-devel, ngraham, #baloo, hurikhan77, lots0logs, 
LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, 
spoorun, abrahams

D24540: [Database] Disable read-ahead to reduce thrashing

2019-10-11 Thread Nathaniel Graham 
ngraham edited the summary of this revision.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D24540

To: hurikhan77, ngraham, #baloo, bruns
Cc: bruns, kde-frameworks-devel, ngraham, #baloo, hurikhan77, lots0logs, 
LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, 
spoorun, abrahams

D24540: [Database] Disable read-ahead to reduce thrashing

2019-10-11 Thread Kai Krakow 
hurikhan77 updated this revision to Diff 67684.
hurikhan77 edited the summary of this revision.
hurikhan77 added a comment.


  Added depend line according to arc docs

REPOSITORY
  R293 Baloo

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D24540?vs=67681=67684

BRANCH
  fixes/readahead

REVISION DETAIL
  https://phabricator.kde.org/D24540

AFFECTED FILES
  src/engine/database.cpp

To: hurikhan77, ngraham, #baloo, bruns
Cc: bruns, kde-frameworks-devel, ngraham, #baloo, hurikhan77, lots0logs, 
LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, 
spoorun, abrahams

D24540: [Database] Disable read-ahead to reduce thrashing

2019-10-11 Thread Kai Krakow 
hurikhan77 updated this revision to Diff 67681.
hurikhan77 added a comment.


  Resend diff with arc

REPOSITORY
  R293 Baloo

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D24540?vs=67629=67681

BRANCH
  fixes/readahead

REVISION DETAIL
  https://phabricator.kde.org/D24540

AFFECTED FILES
  src/engine/database.cpp

To: hurikhan77, ngraham, #baloo, bruns
Cc: bruns, kde-frameworks-devel, ngraham, #baloo, hurikhan77, lots0logs, 
LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, 
spoorun, abrahams

D24540: [Database] Disable read-ahead to reduce thrashing

2019-10-10 Thread Nathaniel Graham 
ngraham added a reviewer: bruns.
ngraham added a subscriber: bruns.
ngraham added a comment.


  @bruns, does this make sense to you?

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D24540

To: hurikhan77, ngraham, #baloo, bruns
Cc: bruns, kde-frameworks-devel, ngraham, #baloo, hurikhan77, lots0logs, 
LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, 
spoorun, abrahams

D24540: [Database] Disable read-ahead to reduce thrashing

2019-10-10 Thread Kai Krakow 
hurikhan77 added a reviewer: Baloo.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D24540

To: hurikhan77, ngraham, #baloo
Cc: kde-frameworks-devel, ngraham, #baloo, hurikhan77, lots0logs, LeGast00n, 
fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, bruns, 
abrahams

D24540: [Database] Disable read-ahead to reduce thrashing

2019-10-10 Thread Kai Krakow 
hurikhan77 added a dependency: D24539: [Database] Rework handling environment 
flags.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D24540

To: hurikhan77, ngraham
Cc: kde-frameworks-devel, ngraham, #baloo, lots0logs, LeGast00n, fbampaloukas, 
GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, bruns, abrahams

D24540: [Database] Disable read-ahead to reduce thrashing

2019-10-10 Thread Kai Krakow 
hurikhan77 created this revision.
hurikhan77 added a reviewer: ngraham.
hurikhan77 added a project: Baloo.
Herald added a project: Frameworks.
Herald added subscribers: Baloo, kde-frameworks-devel.
hurikhan77 requested review of this revision.

REVISION SUMMARY
  Prevent filesystem thrashing in low-mem situations by turning off
  read-ahead. Access patterns of baloo are unlikely to benefit from
  read-ahead and preserving cache is more valuable than reading ahead
  potentially unused data.
  
  In low-memory conditions, the random access patterns of Baloo combined
  with read-ahead tend to dominate the cache. Let Baloo be the slow player
  here, and leave system cache and memory for interactive performance.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D24540

AFFECTED FILES
  src/engine/database.cpp

To: hurikhan77, ngraham
Cc: kde-frameworks-devel, ngraham, #baloo, lots0logs, LeGast00n, fbampaloukas, 
GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, bruns, abrahams