mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix ReorderBufferCheckMemoryLimit() comment.

Browse Source 
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
This commit is contained in:
Masahiko Sawada 2023-08-02 15:01:13 +09:00
parent 3845577cb5
commit 0125c4e21d
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/backend/replication/logical/reorderbuffer.c
View File

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