On Fri, Jul 7, 2023 at 8:31 PM Peter Eisentraut <peter@eisentraut.org> wrote:
> On 21.06.23 10:25, Amit Langote wrote:
> > I realized that the patch for the "other sql/json functions" part is
> > relatively straightforward and has no dependence on the "sql/json
> > query functions" part getting done first. So I've made that one the
> > 0001 patch. The patch I posted in the last email is now 0002, though
> > it only has changes related to changing the order of the patch, so I
> > decided not to change the patch version marker (v1).
>
> (I suggest you change the version number anyway, next time. There are
> plenty of numbers available.)
Will do. :)
> The 0001 patch contains a change to
> doc/src/sgml/keywords/sql2016-02-reserved.txt, which seems
> inappropriate. The additional keywords are already listed in the 2023
> file, and they are not SQL:2016 keywords.
Ah, indeed. Will remove.
> Another thing, I noticed that the SQL/JSON patches in PG16 introduced
> some nonstandard indentation in gram.y. I would like to apply the
> attached patch to straighten this out.
Sounds fine to me.
--
Thanks, Amit Langote
EDB: http://www.enterprisedb.com