diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2017-01-25 09:33:41 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2017-01-25 09:35:00 -0500 |
| commit | aebeb4790c750dc808c1c5afb3cb435116244e36 (patch) | |
| tree | 532565a943fe186d599d075570c523331bf3c0fc /src/backend/parser/analyze.c | |
| parent | 87ecf2d14fa236e894267ef5e702ff08b8965f9d (diff) | |
Remove vestigial resolveUnknown arguments from transformSortClause etc.
There's really no situation where we don't want these unknown-to-text
conversions to happen. The alternative is failure anyway, and the one
caller that was passing "false" did so only because it expected the
case could not arise. Might as well simplify the code.
Discussion: https://postgr.es/m/CAH2L28uwwbL9HUM-WR=hromW1Cvamkn7O-g8fPY2m=_7muJ0oA@mail.gmail.com
Diffstat (limited to 'src/backend/parser/analyze.c')
| -rw-r--r-- | src/backend/parser/analyze.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/backend/parser/analyze.c b/src/backend/parser/analyze.c index f954dc15f0e..0f7659bb6be 100644 --- a/src/backend/parser/analyze.c +++ b/src/backend/parser/analyze.c @@ -1232,7 +1232,6 @@ transformSelectStmt(ParseState *pstate, SelectStmt *stmt) stmt->sortClause, &qry->targetList, EXPR_KIND_ORDER_BY, - true /* fix unknowns */ , false /* allow SQL92 rules */ ); qry->groupClause = transformGroupClause(pstate, @@ -1512,7 +1511,6 @@ transformValuesClause(ParseState *pstate, SelectStmt *stmt) stmt->sortClause, &qry->targetList, EXPR_KIND_ORDER_BY, - true /* fix unknowns */ , false /* allow SQL92 rules */ ); qry->limitOffset = transformLimitClause(pstate, stmt->limitOffset, @@ -1736,7 +1734,6 @@ transformSetOperationStmt(ParseState *pstate, SelectStmt *stmt) sortClause, &qry->targetList, EXPR_KIND_ORDER_BY, - false /* no unknowns expected */ , false /* allow SQL92 rules */ ); /* restore namespace, remove jrte from rtable */ |