diff options
| -rw-r--r-- | src/backend/parser/analyze.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/backend/parser/analyze.c b/src/backend/parser/analyze.c index 65de8610179..3ffa1b63c5d 100644 --- a/src/backend/parser/analyze.c +++ b/src/backend/parser/analyze.c @@ -604,8 +604,16 @@ transformInsertStmt(ParseState *pstate, InsertStmt *stmt, * We have three cases to deal with: DEFAULT VALUES (selectStmt == NULL), * VALUES list, or general SELECT input. We special-case VALUES, both for * efficiency and so we can handle DEFAULT specifications. + * + * The grammar allows attaching ORDER BY, LIMIT, or FOR UPDATE to a + * VALUES clause. If we have any of those, treat it as a general SELECT; + * so it will work, but you can't use DEFAULT items together with those. */ - isGeneralSelect = (selectStmt && selectStmt->valuesLists == NIL); + isGeneralSelect = (selectStmt && (selectStmt->valuesLists == NIL || + selectStmt->sortClause != NIL || + selectStmt->limitOffset != NULL || + selectStmt->limitCount != NULL || + selectStmt->lockingClause != NIL)); /* * If a non-nil rangetable/namespace was passed in, and we are doing |