ForumMagnum / ForumMagnum / 25463223484 / 2

48%

master: 47%

Pull #12497

github

Revert Elasticsearch bulk-indexer rejected filter

Drop the WHERE "rejected" IS NOT TRUE clauses I added to
PostsRepo/CommentsRepo getSearchDocuments and countSearchDocuments.

The prevailing pattern for the ES index is "index every row, filter
at query time": ElasticConfig.ts strips draft, rejected, deleted,
authorIsUnreviewed, retracted, spam, etc. from search results, while
elasticSyncDocument re-indexes every row on update via
getSearchDocumentById, which has no per-row filter. So filtering at
bulk-export time only created an inconsistent state — rejected posts
disappeared after a recreateIndex but came back the next time they
were updated. The query-time filter already handles it.

4953 of 6290 branches covered (78.74%)

Branch coverage included in aggregate %.

157274 of 372522 relevant lines covered (42.22%)

1.93 hits per line