diff options
| author | Dylan Baker <dylan@pnwbakers.com> | 2018-07-31 09:08:48 -0700 |
|---|---|---|
| committer | Anton Khirnov <anton@khirnov.net> | 2021-01-10 11:10:40 +0100 |
| commit | c79196b8b1c5eb99a2f240aaf0069853d798bdd0 (patch) | |
| tree | 5852f43886237c3e245b687cabe8450fc8333fb5 /tests | |
| parent | 47ebd7656057fadb5b627dfc1844bc09dc171ff2 (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 0f3a64ac..dbef6c6c 100644 --- a/tests/db/test_manager.py +++ b/tests/db/test_manager.py @@ -71,7 +71,6 @@ class TestDBManager(utilities.TestCaseClassCleanup): cls.manager = 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 |