diff options
| author | Tom Lane | 2018-02-27 20:56:51 +0000 |
|---|---|---|
| committer | Tom Lane | 2018-02-27 20:56:51 +0000 |
| commit | e98a4de7d23a54a6c861795ad8e920a59613e08b (patch) | |
| tree | 1186827ec34fec9cd236338cca7536a8bce76dcc /src/backend/executor/nodeNamedtuplestorescan.c | |
| parent | c40e20a83ce521b5b5403b08dde05b0f0641d77f (diff) | |
Use the correct tuplestore read pointer in a NamedTuplestoreScan.
Tom Kazimiers reported that transition tables don't work correctly when
they are scanned by more than one executor node. That's because commit
18ce3a4ab allocated separate read pointers for each executor node, as it
must, but failed to make them active at the appropriate times. Repair.
Thomas Munro
Discussion: https://postgr.es/m/20180224034748.bixarv6632vbxgeb%40dewberry.localdomain
Diffstat (limited to 'src/backend/executor/nodeNamedtuplestorescan.c')
| -rw-r--r-- | src/backend/executor/nodeNamedtuplestorescan.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/backend/executor/nodeNamedtuplestorescan.c b/src/backend/executor/nodeNamedtuplestorescan.c index 4d898b1f83e..b260ad25948 100644 --- a/src/backend/executor/nodeNamedtuplestorescan.c +++ b/src/backend/executor/nodeNamedtuplestorescan.c @@ -40,6 +40,7 @@ NamedTuplestoreScanNext(NamedTuplestoreScanState *node) * Get the next tuple from tuplestore. Return NULL if no more tuples. */ slot = node->ss.ss_ScanTupleSlot; + tuplestore_select_read_pointer(node->relation, node->readptr); (void) tuplestore_gettupleslot(node->relation, true, false, slot); return slot; } @@ -116,6 +117,7 @@ ExecInitNamedTuplestoreScan(NamedTuplestoreScan *node, EState *estate, int eflag * The new read pointer copies its position from read pointer 0, which * could be anywhere, so explicitly rewind it. */ + tuplestore_select_read_pointer(scanstate->relation, scanstate->readptr); tuplestore_rescan(scanstate->relation); /* |