diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2017-09-06 10:41:05 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2017-09-06 10:41:05 -0400 |
| commit | 8689e38263af7755b8100203e325a5953ed1e602 (patch) | |
| tree | a059405f6284a8e0818669cb7f1bcc91414d4efb /src/backend/utils/adt/ruleutils.c | |
| parent | 0b554e4e63a4ba4852c01951311713e23acdae02 (diff) | |
| download | postgresql-8689e38263af7755b8100203e325a5953ed1e602.tar.gz postgresql-8689e38263af7755b8100203e325a5953ed1e602.zip | |
Clean up handling of dropped columns in NAMEDTUPLESTORE RTEs.
The NAMEDTUPLESTORE patch piggybacked on the infrastructure for
TABLEFUNC/VALUES/CTE RTEs, none of which can ever have dropped columns,
so the possibility was ignored most places. Fix that, including adding a
specification to parsenodes.h about what it's supposed to look like.
In passing, clean up assorted comments that hadn't been maintained
properly by said patch.
Per bug #14799 from Philippe Beaudoin. Back-patch to v10.
Discussion: https://postgr.es/m/20170906120005.25630.84360@wrigleys.postgresql.org
Diffstat (limited to 'src/backend/utils/adt/ruleutils.c')
| -rw-r--r-- | src/backend/utils/adt/ruleutils.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c index 43646d2c4f8..f9ea7ed771d 100644 --- a/src/backend/utils/adt/ruleutils.c +++ b/src/backend/utils/adt/ruleutils.c @@ -6846,8 +6846,8 @@ get_name_for_var_field(Var *var, int fieldno, case RTE_NAMEDTUPLESTORE: /* - * This case should not occur: a column of a table or values list - * shouldn't have type RECORD. Fall through and fail (most + * This case should not occur: a column of a table, values list, + * or ENR shouldn't have type RECORD. Fall through and fail (most * likely) at the bottom. */ break; |