From: Masahiko Sawada Date: Wed, 2 Aug 2023 06:01:05 +0000 (+0900) Subject: Fix ReorderBufferCheckMemoryLimit() comment. X-Git-Tag: REL_14_9~8 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=2e3741fb5733201f80443ba9b4583f0e3f3df86e;p=thirdparty%2Fpostgresql.git Fix ReorderBufferCheckMemoryLimit() comment. Commit 7259736a6 updated the comment but it was not correct since ReorderBufferLargestStreamableTopTXN() returns only top-level transactions. Reviewed-by: Amit Kapila Discussion: https://postgr.es/m/CAD21AoA9XB7OR86BqvrCe2dMYX%2BZv3-BvVmjF%3DGY2z6jN-kqjg%40mail.gmail.com Backpatch-through: 14 --- diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.c index 721fa652d25..264c253a878 100644 --- a/src/backend/replication/logical/reorderbuffer.c +++ b/src/backend/replication/logical/reorderbuffer.c @@ -3500,8 +3500,8 @@ ReorderBufferCheckMemoryLimit(ReorderBuffer *rb) while (rb->size >= logical_decoding_work_mem * 1024L) { /* - * Pick the largest transaction (or subtransaction) and evict it from - * memory by streaming, if possible. Otherwise, spill to disk. + * Pick the largest transaction and evict it from memory by streaming, + * if possible. Otherwise, spill to disk. */ if (ReorderBufferCanStartStreaming(rb) && (txn = ReorderBufferLargestTopTXN(rb)) != NULL)