From: Tom Lane <tgl@sss.pgh.pa.us>
To: Michael Paquier <michael@paquier.xyz>
Cc: Erik Wienhold <ewie@ewie.name>
Cc: zengman <zengman@halodbtech.com>
Cc: pgsql-hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: Regression with large XML data input
Date: Sun, 28 Dec 2025 22:54:32 -0500
Message-ID: <2744536.1766980472@sss.pgh.pa.us> (raw)
In-Reply-To: <aVHKPeYX3csE1Jwg@paquier.xyz>
References: <aIGknLuc8b8ega2X@paquier.xyz>
<586b74bb-8941-420f-82c4-ba93e5b42b20@ewie.name>
<tencent_2789D934383FA63637EFDB9E@qq.com>
<89bbd852-eea9-42e8-bab6-bb8df59685de@ewie.name>
<1411819.1766685038@sss.pgh.pa.us>
<aVHKPeYX3csE1Jwg@paquier.xyz>
Michael Paquier <michael@paquier.xyz> writes:
> On Thu, Dec 25, 2025 at 12:50:38PM -0500, Tom Lane wrote:
>> That would be my preferred answer, but after a quick census of the
>> buildfarm I'm not sure we can get away with it. The animals that
>> are showing these warnings are all running RHEL 10 or clones of it,
>> which is going to be in-support till 2035 or so. Maybe Red Hat
>> will upgrade to a newer libxml2 version than whatever they chose
>> for RHEL 10.0, but I would not count on it.
> Hmm. Having to deal with these warnings until 2035 kind of sucks. Do
> you think that it be worth forcing the use of
> -Wno-deprecated-declarations only for versions older than 2.13.3 in
> the Makefiles where we could see these warnings?
It seems pretty painful to identify the libxml2 version during
configure, do you have an idea how to do that? (And how about
on the meson side?)
> As a whole, this kind of enforcement gives me mixed feelings, because
> it would also mean that we would miss what looks like legit
> deprecation warnings that we had better look at and study, even for
> LLVM. At least not enforcing any policy lets us know about these
> issues.
Right. I feel very itchy about adding -Wno-deprecated-declarations
with no plan for getting rid of it again. Seeing that libxml2 seems
to be a moving target again, it seems like that would eventually bite
us on the rear.
[ thinks... ] Actually, I believe we do have precedents in configure
for checking to see if a construct produces a warning, so we could
check to see if a reference to xmlKeepBlanksDefault draws one, and
only add -Wno-deprecated-declarations if it does. But I'm not sure
how to do that in meson.
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, michael@paquier.xyz, ewie@ewie.name, zengman@halodbtech.com, pgsql-hackers@lists.postgresql.org
Subject: Re: Regression with large XML data input
In-Reply-To: <2744536.1766980472@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