From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 1CBD41F9FD for ; Mon, 15 Mar 2021 11:58:27 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 00/35] memoize inbox creations Date: Mon, 15 Mar 2021 12:57:51 +0100 Message-Id: <20210315115826.17591-1-e@80x24.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: This speeds up a bunch of tests by a few dozen to hundreds of milliseconds. Part of it is due to disabling fsync/fdatasync in a few more places. Users with slower TMPDIR will notice more, and users with TMPDIR pointed to tmpfs will notice it least. Overall it's not hugely significant, but the diffstat and LoC reduction still makes it nice even if there were no performance improvements. Eric Wong (35): test_common: add create_inbox helper sub t/lei_xsearch: use create_inbox test_common: minor simplifications to setup_public_inboxes t/imapd-tls: switch to create_inbox t/www_altid: use create_inbox t/xcpdb-reshard: use create_inbox t/v2dupindex: create_inbox t/admin: switch to create_inbox t/html_index: remove now-worthless test t/plack: use create_inbox t/psgi_attach: convert to create_inbox t/httpd: convert to create_inbox t/convert-compact: create_inbox t/altid: use create_inbox t/psgi_mount: switch to create_inbox t/feed: switch to create_inbox t/psgi_bad_mids: use create_inbox t/psgi_multipart_not: use create_inbox t/psgi_scan_all: create_inbox lots saved t/psgi_v2: create_inbox t/imapd: create_inbox (minor) t/solver_git: use create_inbox t/cgi: create_inbox t/edit: switch to create_inbox t/inbox_idle: switch to create_inbox t/lei_xsearch: use create_inbox t/indexlevels-mirror: use create_inbox t/multi-mid: use create_inbox t/psgi_search: use create_inbox t/miscsearch: use create_inbox t/nntpd-tls: use create_inbox t/nntpd: use create_inbox t/thread-index-gap: create_inbox t/altid_v2: create_inbox t/*: disable fsync on tests were create_inbox isn't worth it MANIFEST | 2 +- Makefile.PL | 3 +- lib/PublicInbox/TestCommon.pm | 67 ++++++++++++++--- t/admin.t | 38 ++++------ t/altid.t | 26 ++----- t/altid_v2.t | 48 +++++------- t/cgi.t | 84 +++++++++------------ t/convert-compact.t | 48 +++++------- t/data-gen/.gitignore | 2 + t/edit.t | 34 ++++----- t/extsearch.t | 2 + t/feed.t | 53 ++++---------- t/html_index.t | 56 -------------- t/httpd.t | 36 +++------ t/imapd-tls.t | 38 +++------- t/imapd.t | 66 ++++++++--------- t/inbox_idle.t | 31 ++++---- t/indexlevels-mirror.t | 24 +++--- t/lei-q-remote-import.t | 20 ++--- t/lei_xsearch.t | 56 +++++--------- t/miscsearch.t | 28 +++---- t/multi-mid.t | 34 +++------ t/nntpd-tls.t | 52 ++++--------- t/nntpd.t | 102 ++++++++++++-------------- t/plack.t | 69 +++++++---------- t/psgi_attach.t | 56 ++++++-------- t/psgi_bad_mids.t | 46 ++++-------- t/psgi_mount.t | 35 ++++----- t/psgi_multipart_not.t | 35 +++------ t/psgi_scan_all.t | 48 ++++-------- t/psgi_search.t | 37 +++------- t/psgi_v2.t | 134 ++++++++++++++++------------------ t/purge.t | 1 + t/replace.t | 1 + t/solver_git.t | 88 +++++++++++----------- t/thread-index-gap.t | 24 +++--- t/v1reindex.t | 1 + t/v2-add-remove-add.t | 1 + t/v2dupindex.t | 76 +++++++++---------- t/v2mirror.t | 1 + t/v2reindex.t | 1 + t/v2writable.t | 1 + t/watch_filter_rubylang.t | 2 + t/watch_maildir_v2.t | 1 + t/www_altid.t | 47 +++++------- t/xcpdb-reshard.t | 54 ++++++-------- 46 files changed, 692 insertions(+), 1017 deletions(-) create mode 100644 t/data-gen/.gitignore delete mode 100644 t/html_index.t