[pgpool-hackers: 3253] Re: [pgpool-committers: 5093] pgpool: Import PostgreSQL 11 beta3 parser.

[Yugo Nagata]

Hi Peng,

I found the import of PG11 parser is insufficient so that the
native replication mode can not rewrite queries including GROUPS
in frame clauses.
https://www.postgresql.org/docs/11/sql-select.html

For example,

test=# \d test_tbl;
              Table "public.test_tbl"
 Column |  Type   | Collation | Nullable | Default 
--------+---------+-----------+----------+---------
 i      | integer |           |          | 

test=# \d result 
                         Table "public.result"
 Column |            Type             | Collation | Nullable | Default 
--------+-----------------------------+-----------+----------+---------
 t      | timestamp without time zone |           |          | 
 sum    | integer                     |           |          | 

test=#  select now(), sum(i) over (order by i groups between 1 preceding and 1 following) from test_tbl;
              now              | sum 
-------------------------------+-----
 2019-02-26 17:50:40.262925+09 |   3
 2019-02-26 17:50:40.262925+09 |   6
 2019-02-26 17:50:40.262925+09 |   5
(3 rows)

test=# insert into result select now(), sum(i) over (order by i groups between 1 preceding and 1 following) from test_tbl;
ERROR:  syntax error at or near "PRECEDING"

According to the log, this query is rewritten as following:

INSERT INTO "result" SELECT "pg_catalog"."timestamptz"('2019-02-26 17:51:00.749663+09'::text),
 "sum"("i") OVER ( ORDER BY "i" BETWEEN 1 PRECEDING AND 1 FOLLOWING) FROM "test_tbl"

As you see, GROUPS is omitted in the rewritten query because the codes to handle this 
is missing in outfuncs.c. Similarly, I guess EXCLUDE in frame clauses is not properly
handled, neither.

Regards,

On Tue, 28 Aug 2018 09:21:02 +0000
Bo Peng <pengbo at sraoss.co.jp> wrote:

> Import PostgreSQL 11 beta3 parser.
> 
> Branch
> ------
> master
> 
> Details
> -------
> https://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=8b43ac3b956022447b41b744baf2260dca20bba1
> 
> Modified Files
> --------------
> src/include/parser/gramparse.h        |    4 +-
> src/include/parser/keywords.h         |    4 +-
> src/include/parser/kwlist.h           |   12 +-
> src/include/parser/makefuncs.h        |    6 +-
> src/include/parser/nodes.h            |   16 +-
> src/include/parser/parsenodes.h       |  182 ++--
> src/include/parser/parser.h           |    8 +-
> src/include/parser/pg_class.h         |   84 +-
> src/include/parser/pg_config_manual.h |   20 +-
> src/include/parser/pg_list.h          |    7 +-
> src/include/parser/pg_trigger.h       |   42 +-
> src/include/parser/pg_wchar.h         |   17 +-
> src/include/parser/primnodes.h        |   37 +-
> src/include/parser/scanner.h          |    4 +-
> src/include/parser/scansup.h          |    4 +-
> src/include/parser/stringinfo.h       |   12 +-
> src/include/parser/value.h            |   10 +-
> src/parser/copyfuncs.c                |  163 +++-
> src/parser/gram.h                     | 1622 +++++++++++++++++----------------
> src/parser/gram.y                     | 1076 +++++++++++++++-------
> src/parser/keywords.c                 |    4 +-
> src/parser/list.c                     |   66 +-
> src/parser/makefuncs.c                |   25 +-
> src/parser/nodes.c                    |    4 +-
> src/parser/outfuncs.c                 |  113 ++-
> src/parser/parser.c                   |   29 +-
> src/parser/scan.l                     |   18 +-
> src/parser/scansup.c                  |    6 +-
> src/parser/snprintf.c                 |   14 +-
> src/parser/stringinfo.c               |   25 +-
> src/parser/value.c                    |    4 +-
> src/parser/wchar.c                    |   12 +-
> src/protocol/pool_proto_modules.c     |    2 +-
> 33 files changed, 2192 insertions(+), 1460 deletions(-)
> 


-- 
Yugo Nagata <nagata at sraoss.co.jp>