pgsql: Add location field to DefElem - Mailing list pgsql-committers

From Peter Eisentraut
Subject pgsql: Add location field to DefElem
Date
Msg-id E1bhT5R-00082M-Ma@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Add location field to DefElem

Add a location field to the DefElem struct, used to parse many utility
commands.  Update various error messages to supply error position
information.

To propogate the error position information in a more systematic way,
create a ParseState in standard_ProcessUtility() and pass that to
interested functions implementing the utility commands.  This seems
better than passing the query string and then reassembling a parse state
ad hoc, which violates the encapsulation of the ParseState type.

Reviewed-by: Pavel Stehule <pavel.stehule@gmail.com>

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/49eb0fd0972d14014dd3533b1f1bf8c94c899883

Modified Files
--------------
contrib/file_fdw/file_fdw.c                    |  16 +-
src/backend/access/common/reloptions.c         |   2 +-
src/backend/catalog/aclchk.c                   |   8 +-
src/backend/commands/aggregatecmds.c           |   7 +-
src/backend/commands/collationcmds.c           |   5 +-
src/backend/commands/copy.c                    |  93 ++++++----
src/backend/commands/dbcommands.c              |  61 +++---
src/backend/commands/define.c                  |   9 -
src/backend/commands/explain.c                 |   8 +-
src/backend/commands/extension.c               |  25 ++-
src/backend/commands/functioncmds.c            |  57 +++---
src/backend/commands/sequence.c                |  36 ++--
src/backend/commands/tsearchcmds.c             |   8 +-
src/backend/commands/typecmds.c                |   8 +-
src/backend/commands/user.c                    |  41 ++--
src/backend/commands/view.c                    |   4 +-
src/backend/nodes/copyfuncs.c                  |   1 +
src/backend/nodes/equalfuncs.c                 |   2 +
src/backend/nodes/makefuncs.c                  |   6 +-
src/backend/nodes/outfuncs.c                   |   1 +
src/backend/nodes/readfuncs.c                  |   1 +
src/backend/parser/gram.y                      | 248 ++++++++++++-------------
src/backend/parser/parse_utilcmd.c             |   5 +-
src/backend/replication/logical/logicalfuncs.c |   2 +-
src/backend/replication/repl_gram.y            |  16 +-
src/backend/tcop/utility.c                     |  64 ++++---
src/include/commands/collationcmds.h           |   2 +-
src/include/commands/copy.h                    |   7 +-
src/include/commands/dbcommands.h              |   4 +-
src/include/commands/defrem.h                  |  13 +-
src/include/commands/explain.h                 |   3 +-
src/include/commands/extension.h               |   4 +-
src/include/commands/sequence.h                |   5 +-
src/include/commands/typecmds.h                |   2 +-
src/include/commands/user.h                    |   3 +-
src/include/nodes/makefuncs.h                  |   4 +-
src/include/nodes/parsenodes.h                 |   1 +
src/include/utils/acl.h                        |   3 +-
38 files changed, 438 insertions(+), 347 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Doc: small improvements for documentation about VACUUM freezing.
Next
From: Tom Lane
Date:
Subject: pgsql: Add a HINT for client-vs-server COPY failure cases.