diff options
| author | Dylan Baker <dylan@pnwbakers.com> | 2018-07-31 09:08:48 -0700 |
|---|---|---|
| committer | Patrick Totzke <patricktotzke@gmail.com> | 2019-08-15 20:44:32 +0100 |
| commit | e7e0c52db9093a9ecd9dcaa0766e66515a546a75 (patch) | |
| tree | d669b366f43cc3cdbc7c57020deb8230ac69158a /tests | |
| parent | cd44abdc4df887e1e9ed70e510560fc26e62f97e (diff) | |
db/manager: Drop async method
As far as I can tell using a separate process doesn't actually improve
performance, it makes it worse. The work that we're passing off to the
separate function isn't necessarily work that's well suited to being
handed off, there isn't a lot of computation and the objects that need
to be passed across the pipe are fairly large (at least when considering
a pipe). Converting the function to a generator gives better performance
and simplifies the implementation.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/db/test_manager.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tests/db/test_manager.py b/tests/db/test_manager.py index e675aed4..f7a47a58 100644 --- a/tests/db/test_manager.py +++ b/tests/db/test_manager.py @@ -37,7 +37,6 @@ class TestDBManager(utilities.TestCaseClassCleanup): cls.manager = DBManager(cls.dbpath) # clean up temporary database - cls.addClassCleanup(cls.manager.kill_search_processes) cls.addClassCleanup(shutil.rmtree, cls.dbpath) # let global settings manager read our temporary notmuch config |