From: Tom Lane <tgl@sss.pgh.pa.us>
To: Chapman Flack <chap@anastigmatix.net>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Cc: Markus Winand <markus.winand@winand.at>
Cc: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Michael Paquier <michael@paquier.xyz>
Cc: Pavel Stehule <pavel.stehule@gmail.com>
Subject: Re: The two "XML Fixes" patches still in need of review
Date: Sat, 23 Mar 2019 17:05:14 -0400
Message-ID: <12427.1553375114@sss.pgh.pa.us> (raw)
In-Reply-To: <5C81F8C0.6090901@anastigmatix.net>
References: <5C81F8C0.6090901@anastigmatix.net>
Chapman Flack <chap@anastigmatix.net> writes:
> Alvaro has committed two of the patches in this CF entry[1], but the
> remaining two have yet to attract review.
> This message contains only those two, just as before[2] except rebased
> over Alvaro's commits of the others.
Just to update this thread --- per the other thread at
https://postgr.es/m/CAN-V+g-6JqUQEQZ55Q3toXEN6d5Ez5uvzL4VR+8KtvJKj31taw@mail.gmail.com
I've now pushed a somewhat-adjusted version of the XML-content fix
patch. The documentation patch needs some small rebasing to apply
after that one instead of before it.
I'm not going to touch the documentation patch myself; I don't know
enough about XML to review it competently. I do have a small suggestion
though, which is that the large "Limits and Compatibility" section
you added doesn't really seem to me to belong where you put it.
Perhaps it'd make sense under the XML section in datatype.sgml,
but I think I might lean to making it a new section in Appendix D
(SQL Conformance).
regards, tom lane
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, chap@anastigmatix.net, pgsql-hackers@lists.postgresql.org, markus.winand@winand.at, alvherre@2ndquadrant.com, michael@paquier.xyz, pavel.stehule@gmail.com
Subject: Re: The two "XML Fixes" patches still in need of review
In-Reply-To: <12427.1553375114@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox