agora inbox for [email protected]
help / color / mirror / Atom feedFrom: Alvaro Herrera <[email protected]>
Subject: [PATCH 8/8] reverse arguments .. isn't the other order a bug?
Date: Thu, 12 Mar 2020 18:33:02 -0300
---
src/backend/executor/nodeIncrementalSort.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/src/backend/executor/nodeIncrementalSort.c b/src/backend/executor/nodeIncrementalSort.c
index 909d2df53f..bb88fca207 100644
--- a/src/backend/executor/nodeIncrementalSort.c
+++ b/src/backend/executor/nodeIncrementalSort.c
@@ -653,7 +653,7 @@ ExecIncrementalSort(PlanState *pstate)
&node->incsort_info.fullsortGroupInfo,
fullsort_state);
- SO_printf("Setting execution_status to INCSORT_READFULLSORT (final tuple) \n");
+ SO_printf("Setting execution_status to INCSORT_READFULLSORT (final tuple)\n");
node->execution_status = INCSORT_READFULLSORT;
break;
}
@@ -713,7 +713,8 @@ ExecIncrementalSort(PlanState *pstate)
* configuring sorting bound.
*/
SO2_printf("Changing bound_Done from %ld to %ld\n",
- Min(node->bound, node->bound_Done + nTuples), node->bound_Done);
+ node->bound_Done,
+ Min(node->bound, node->bound_Done + nTuples));
node->bound_Done = Min(node->bound, node->bound_Done + nTuples);
}
@@ -721,7 +722,8 @@ ExecIncrementalSort(PlanState *pstate)
* Once we find changed prefix keys we can complete the
* sort and begin reading out the sorted tuples.
*/
- SO1_printf("Sorting fullsort tuplesort with %ld tuples\n", nTuples);
+ SO1_printf("Sorting fullsort tuplesort with %ld tuples\n",
+ nTuples);
tuplesort_performsort(fullsort_state);
if (pstate->instrument != NULL)
@@ -882,7 +884,8 @@ ExecIncrementalSort(PlanState *pstate)
* in configuring sorting bound.
*/
SO2_printf("Changing bound_Done from %ld to %ld\n",
- Min(node->bound, node->bound_Done + nTuples), node->bound_Done);
+ node->bound_Done,
+ Min(node->bound, node->bound_Done + nTuples));
node->bound_Done = Min(node->bound, node->bound_Done + nTuples);
}
}
--
2.20.1
--jI8keyz6grp/JLjh--
view thread (83+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected]
Subject: Re: [PATCH 8/8] reverse arguments .. isn't the other order a bug?
In-Reply-To: <no-message-id-1882510@localhost>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox