From: Nathan Bossart <nathandbossart@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Fujii Masao <masao.fujii@oss.nttdata.com>
Cc: Andy Fan <zhihuifan1213@163.com>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: pgbench error: (setshell) of script 0; execution of meta-command failed
Date: Tue, 14 Jan 2025 10:29:53 -0600
Message-ID: <Z4aRAdGdw8GiU7ba@nathan> (raw)
In-Reply-To: <570943.1736808714@sss.pgh.pa.us>
References: <87msfz5qv2.fsf@163.com>
<87ikqn5em7.fsf@163.com>
<e03fc711-8c48-4fb8-ba94-0862c920c30f@oss.nttdata.com>
<b0afe844-84b7-4154-ad2c-6763ff84d876@oss.nttdata.com>
<2422778.1736521065@sss.pgh.pa.us>
<2425050.1736522451@sss.pgh.pa.us>
<127979.1736622253@sss.pgh.pa.us>
<Z4U5j_FWpNLjtdU8@nathan>
<570943.1736808714@sss.pgh.pa.us>
On Mon, Jan 13, 2025 at 05:51:54PM -0500, Tom Lane wrote:
> It's fair to worry about this, but I don't think my testing that would
> prove a lot. AFAICS, whether somebody runs into trouble would depend
> on many factors like their specific build process and what versions of
> which packages they have installed.
>
> In any case, I think we have a very limited amount of wiggle room.
> We definitely cannot change libpq's ABI without provoking howls of
> anguish.
Fair point.
> I have been wondering whether it would help to add something like
> this at the end of port/pqsignal.c in the back branches:
>
> #ifdef FRONTEND
> #undef pqsignal
>
> /* ABI-compatibility wrapper */
> pqsigfunc
> pqsignal(int signo, pqsigfunc func)
> {
> return pqsignal_fe(signo, func);
> }
> #endif
>
> (plus or minus an extern or so, but you get the idea). The point of
> this is that compiling against old headers and then linking against
> newer libpgport.a would still work. It does nothing however for the
> reverse scenario of compiling against new headers and then linking
> against old libpgport.a. So I haven't persuaded myself that it's
> worth the trouble -- but I'm happy to include it if others think
> it would help.
After sleeping on it, I think I agree that the extra gymnastics aren't
worth it to partially fix something that wasn't supported anyway. But I'm
not mortally opposed to it if someone feels strongly that it should be
included.
--
nathan
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: nathandbossart@gmail.com, tgl@sss.pgh.pa.us, masao.fujii@oss.nttdata.com, zhihuifan1213@163.com, pgsql-hackers@lists.postgresql.org
Subject: Re: pgbench error: (setshell) of script 0; execution of meta-command failed
In-Reply-To: <Z4aRAdGdw8GiU7ba@nathan>
* 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