sqlite: PRAGMA optimize on close

As recommended by SQLite documentation[1]:

  To achieve the best long-term query performance without the need
  to do a detailed engineering analysis of the application schema
  and SQL, it is recommended that applications run "PRAGMA optimize"
  (with no arguments) just before closing each database connection.

Hopefully that works for our use cases and can make things
faster for us.

[1] https://www.sqlite.org/pragma.html#pragma_optimize

1 files changed, 11 insertions, 0 deletions

diff --git a/lib/PublicInbox/V2Writable.pm b/lib/PublicInbox/V2Writable.pm
index fcd7ffe2..d04cdda6 100644
--- a/lib/PublicInbox/V2Writable.pm
+++ b/lib/PublicInbox/V2Writable.pm
@@ -622,7 +622,18 @@ sub done {
                 my $m = $err ? 'rollback' : 'commit';
                 eval { $mm->{dbh}->$m };
                 $err .= "msgmap $m: $@\n" if $@;
+                eval { $mm->{dbh}->do('PRAGMA optimize') };
+                $err .= "msgmap optimize: $@\n" if $@;
         }
+        if ($self->{oidx} && $self->{oidx}->{dbh}) {
+                if ($err) {
+                        eval { $self->{oidx}->rollback_lazy };
+                        $err .= "overview rollback: $@\n" if $@;
+                }
+                eval { $self->{oidx}->{dbh}->do('PRAGMA optimize') };
+                $err .= "overview optimize: $@\n" if $@;
+        }
+
         my $shards = delete $self->{idx_shards};
         if ($shards) {
                 for (@$shards) {