[pgpool-committers: 3030] pgpool: Properly process empty query (all comments)

Tatsuo Ishii ishii at postgresql.org
Wed Feb 17 18:07:43 JST 2016


Properly process empty query (all comments)

Per bug#154. Pgpool-II recognizes empty query, for example "/* DBD::Pg
ping test v3.5.3 */" (note that no ';') as an error and emits message
" Unable to parse the query". This is because raw_parser() cannot
distinguish an empty query from an error query due to the poor API
design. To fix this new out parameter "*error" added to the
function. If an error occurred, the variable is set to true and caller
can know the reason why raw_parser() returns NIL is because of error
or empty query. See [pgpool-hackers: 1359] for additional information.

Branch
------
V3_5_STABLE

Details
-------
http://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=bb95f4052dbf2c531ad2b2b5b0c0760f6bcb99bf

Modified Files
--------------
src/include/parser/parser.h                        |  4 ++--
src/parser/parser.c                                |  9 +++++++-
src/protocol/pool_process_query.c                  |  5 +++--
src/protocol/pool_proto_modules.c                  | 25 +++++++++++++---------
.../tests/010.rewrite_timestamp/timestamp/main.c   |  3 ++-
5 files changed, 30 insertions(+), 16 deletions(-)



More information about the pgpool-committers mailing list