diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2022-08-12 12:05:50 +0200 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2022-08-12 12:05:50 +0200 |
| commit | 455d254d22665eb7af3b714435f3fecc63696cc7 (patch) | |
| tree | 3fed7037832968b7a77391721e86f1d1622778f1 /src/backend/commands/copy.c | |
| parent | e1878cf94134429bf951b41ed523363c2fa2d6b4 (diff) | |
Reject MERGE in CTEs and COPY
The grammar added for MERGE inadvertently made it accepted syntax in
places that were not prepared to deal with it -- namely COPY and inside
CTEs, but invoking these things with MERGE currently causes assertion
failures or weird misbehavior in non-assertion builds. Protect those
places by checking for it explicitly until somebody decides to implement
it.
Reported-by: Alexey Borzov <borz_off@cs.msu.su>
Discussion: https://postgr.es/m/17579-82482cd7b267b862@postgresql.org
Diffstat (limited to 'src/backend/commands/copy.c')
| -rw-r--r-- | src/backend/commands/copy.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/backend/commands/copy.c b/src/backend/commands/copy.c index e2870e3c11c..2e90563333e 100644 --- a/src/backend/commands/copy.c +++ b/src/backend/commands/copy.c @@ -273,6 +273,12 @@ DoCopy(ParseState *pstate, const CopyStmt *stmt, { Assert(stmt->query); + /* MERGE is allowed by parser, but unimplemented. Reject for now */ + if (IsA(stmt->query, MergeStmt)) + ereport(ERROR, + errcode(ERRCODE_FEATURE_NOT_SUPPORTED), + errmsg("MERGE not supported in COPY")); + query = makeNode(RawStmt); query->stmt = stmt->query; query->stmt_location = stmt_location; |