diff options
author | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-02-22 21:17:53 +0000 |
---|---|---|
committer | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-02-28 18:45:35 +0000 |
commit | ebc3f825c1eb95399c575fff816180a6e4fffeb6 (patch) | |
tree | 0d71e915376dbf05b251881eeef00cc2d8e36156 /lib/PublicInbox/Inbox.pm | |
parent | 126d94b10886b02a3936f1429cab1eaea21b6025 (diff) | |
download | public-inbox-ebc3f825c1eb95399c575fff816180a6e4fffeb6.tar.gz |
Relying more on Xapian requires retrying reopens in more places to ensure it does not fall down and show errors to the user.
Diffstat (limited to 'lib/PublicInbox/Inbox.pm')
-rw-r--r-- | lib/PublicInbox/Inbox.pm | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/PublicInbox/Inbox.pm b/lib/PublicInbox/Inbox.pm index f000a950..54a6eb32 100644 --- a/lib/PublicInbox/Inbox.pm +++ b/lib/PublicInbox/Inbox.pm @@ -254,9 +254,11 @@ sub msg_by_mid ($$;$) { my ($self, $mid, $ref) = @_; my $srch = search($self) or return msg_by_path($self, mid2path($mid), $ref); - my $smsg = $srch->lookup_skeleton($mid) or return; - $smsg->load_expand; - msg_by_smsg($self, $smsg, $ref); + my $smsg; + $srch->retry_reopen(sub { + $smsg = $srch->lookup_skeleton($mid) and $smsg->load_expand; + }); + $smsg ? msg_by_smsg($self, $smsg, $ref) : undef; } 1; |