Re: Regression with large XML data input

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Erik Wienhold <ewie(at)ewie(dot)name>, zengman <zengman(at)halodbtech(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Regression with large XML data input
Date: 2025-12-29 03:54:32
Message-ID: 2744536.1766980472@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Michael Paquier <michael(at)paquier(dot)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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Henson Choi 2025-12-29 04:41:34 Re: RFC: PostgreSQL Storage I/O Transformation Hooks
Previous Message Michael Paquier 2025-12-29 03:51:23 Re: [Proposal] Expose internal MultiXact member count function for efficient monitoring