Fix ReorderBufferCheckMemoryLimit() comment.
authorMasahiko Sawada <[email protected]>
Wed, 2 Aug 2023 06:01:13 +0000 (15:01 +0900)
committerMasahiko Sawada <[email protected]>
Wed, 2 Aug 2023 06:01:13 +0000 (15:01 +0900)
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
Back-through: 14

src/backend/replication/logical/reorderbuffer.c

index 26d252bd87571c616f2b34dee666c95649086def..87a4d2a24b7ac2259cd264b39c0070639c440d51 100644 (file)
@@ -3602,8 +3602,8 @@ ReorderBufferCheckMemoryLimit(ReorderBuffer *rb)
            rb->size > 0))
    {
        /*
-        * Pick the largest transaction (or subtransaction) and evict it from
-        * memory by , if possible.  Otherwise, spill to disk.
+        * Pick the largest transaction and evict it from memory by ,
+        * if possible.  Otherwise, spill to disk.
         */
        if (ReorderBufferCanStart(rb) &&
            (txn = ReorderBufferLargestStreamableTopTXN(rb)) != NULL)