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-ASN: 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 8907520383 for ; Thu, 23 May 2019 09:37:05 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 03/26] doc: document the reason for --no-renumber Date: Thu, 23 May 2019 09:36:41 +0000 Message-Id: <20190523093704.18367-4-e@80x24.org> In-Reply-To: <20190523093704.18367-1-e@80x24.org> References: <20190523093704.18367-1-e@80x24.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: We're going to need copydatabase, too --- Documentation/public-inbox-v1-format.pod | 4 ++++ Documentation/public-inbox-v2-format.pod | 4 ++++ script/public-inbox-compact | 2 ++ 3 files changed, 10 insertions(+) diff --git a/Documentation/public-inbox-v1-format.pod b/Documentation/public-inbox-v1-format.pod index 2a6b8d3..3b0e70e 100644 --- a/Documentation/public-inbox-v1-format.pod +++ b/Documentation/public-inbox-v1-format.pod @@ -134,6 +134,10 @@ Since SCHEMA_VERSION 15 and the development of the v2 format, the "overview" DB also exists in the xapian directory for v1 repositories. See L +Our use of the L requires Xapian document IDs to +remain stable. Thus, use of L and +L require the use of C<--no-renumber> switch. + =item $GIT_DIR/ssoma.index This file is no longer used or created by public-inbox, but it is diff --git a/Documentation/public-inbox-v2-format.pod b/Documentation/public-inbox-v2-format.pod index 7dfe329..bc58074 100644 --- a/Documentation/public-inbox-v2-format.pod +++ b/Documentation/public-inbox-v2-format.pod @@ -117,6 +117,10 @@ Rotational storage devices are NOT recommended for indexing of large mail archives; but are fine for backup and usable for small instances. +Our use of the L requires Xapian document IDs to +remain stable. Thus, use of L and +L require the use of C<--no-renumber> switch. + =head2 OVERVIEW DB Towards the end of v2 development, it became apparent Xapian did diff --git a/script/public-inbox-compact b/script/public-inbox-compact index d22e403..395eec3 100755 --- a/script/public-inbox-compact +++ b/script/public-inbox-compact @@ -51,6 +51,8 @@ sub commit_changes ($$$) { $im->lock_release; remove_tree("$old/old") or die "failed to remove $old/old: $!\n"; } + +# we rely on --no-renumber to keep docids synched to NNTP my @compact = qw(xapian-compact --no-renumber); if ($v == 2) { require PublicInbox::V2Writable; -- EW