Re: The two "XML Fixes" patches still in need of review - Mailing list pgsql-hackers

From Chapman Flack
Subject Re: The two "XML Fixes" patches still in need of review
Date
Msg-id 5C96B141.5050802@anastigmatix.net
Whole thread Raw
In response to Re: The two "XML Fixes" patches still in need of review  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: The two "XML Fixes" patches still in need of review
Re: The two "XML Fixes" patches still in need of review
List pgsql-hackers
On 03/23/19 17:05, Tom Lane wrote:

> 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.

Will do.

> 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).

Sounds like the option (4) I proposed back in [1]. I suppose it won't
be much trouble to move.

> I'm not going to touch the documentation patch myself; I don't know
> enough about XML to review it competently.

I would be happy even to see it reviewed for style, and if anybody's
willing to look at the technical content even if not feeling ideally
prepared, I'd be happy to respond or give references for any points
in question.

It does fix a number of actual bugs in the existing documentation, so
while it would be ideal to apply a higher standard of technical review
to this patch, it'd be less ideal to have existing buggy doc lingering on
because nobody feels up to reviewing at that ideal higher standard.

Regards,
-Chap

[1] https://www.postgresql.org/message-id/5C298BCB.4060704%40anastigmatix.net


pgsql-hackers by date:

Previous
From: legrand legrand
Date:
Subject: RE: Planning counters in pg_stat_statements (using pgss_store)
Next
From: Tom Lane
Date:
Subject: Re: Fix XML handling with DOCTYPE