umputun / remark42 / 28705580352

84%

master: 84%

Pull #2109

github

Cache per-user flag lookups in comment listings

alterComment issued two engine.Flag calls (Blocked, then Verified) for
every comment, so a listing of N comments triggered up to 2N BoltDB read
transactions even when many comments shared the same author. Find,
FindSince, User and Last all funnel through it.

Add a userFlagCache that memoises blocked/verified results by site and
user for the duration of a single listing, so repeated authors are
looked up once. alterComment keeps its signature for single-comment
callers (Get) by using a fresh cache; the batch paths share one.

35 of 37 new or added lines in 1 file covered. (94.59%)

6434 of 7618 relevant lines covered (84.46%)

36.93 hits per line