diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2015-05-17 21:22:12 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2015-05-17 21:22:12 -0400 |
| commit | 424661913c06af76a46fdff9cc24cc57abf14fb3 (patch) | |
| tree | 5d27c01f805aaf3cce92844344e51f37670573a4 /src/backend/nodes/copyfuncs.c | |
| parent | b14cf229f4bd7238be2e31d873dc5dd241d3871e (diff) | |
| download | postgresql-424661913c06af76a46fdff9cc24cc57abf14fb3.tar.gz postgresql-424661913c06af76a46fdff9cc24cc57abf14fb3.zip | |
Fix failure to copy IndexScan.indexorderbyops in copyfuncs.c.
This oversight results in a crash at executor startup if the plan has
been copied. outfuncs.c was missed as well.
While we could probably have taught both those files to cope with the
originally chosen representation of an Oid array, it would have been
painful, not least because there'd be no easy way to verify the array
length. An Oid List is far easier to work with. And AFAICS, there is
no particular notational benefit to using an array rather than a list
in the existing parts of the patch either. So just change it to a list.
Error in commit 35fcb1b3d038a501f3f4c87c05630095abaaadab, which is new,
so no need for back-patch.
Diffstat (limited to 'src/backend/nodes/copyfuncs.c')
| -rw-r--r-- | src/backend/nodes/copyfuncs.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/backend/nodes/copyfuncs.c b/src/backend/nodes/copyfuncs.c index fa7d2865c1e..d36be3c8068 100644 --- a/src/backend/nodes/copyfuncs.c +++ b/src/backend/nodes/copyfuncs.c @@ -381,6 +381,7 @@ _copyIndexScan(const IndexScan *from) COPY_NODE_FIELD(indexqualorig); COPY_NODE_FIELD(indexorderby); COPY_NODE_FIELD(indexorderbyorig); + COPY_NODE_FIELD(indexorderbyops); COPY_SCALAR_FIELD(indexorderdir); return newnode; |