From: Christoph Berg <myon@debian.org>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Michael Banck <mbanck@gmx.net>
Cc: Tommy Pavlicek <tommypav122@gmail.com>
Cc: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Cc: pgsql-hackers@lists.postgresql.org
Cc: jian.universality@gmail.com
Subject: Better error reporting from extension scripts (Was: Extend ALTER OPERATOR)
Date: Fri, 27 Sep 2024 14:17:15 +0200
Message-ID: <ZvaiS57G10dITjQm@msg.df7cb.de> (raw)
In-Reply-To: <325394.1727365907@sss.pgh.pa.us>
References: <CAEhP-W8MjoA0yDCipp=FgZvoQA-OVjQws3-XO6bpCWXunO8t8A@mail.gmail.com>
<447166.1696969927@sss.pgh.pa.us>
<CAEhP-W_1bMTLan2LtzS+xr=b-Wep56gw+OoirABQ8b2aOw+5yg@mail.gmail.com>
<ZTfL1G1fBPQHVPWV@msg.df7cb.de>
<560086.1698160614@sss.pgh.pa.us>
<ZTfjXzOKFI9AawJF@msg.df7cb.de>
<570833.1698162135@sss.pgh.pa.us>
<66f33840.050a0220.1f9e60.7fb9@mx.google.com>
<ZvV1ClhnbJLCz7Sm@msg.df7cb.de>
<325394.1727365907@sss.pgh.pa.us>
Re: Tom Lane
> So the first part of that is great, but if your script file is
> large you probably won't be happy about having the whole thing
> repeated in the "QUERY" field. So this needs some work on
> user-friendliness.
Does this really have to be addressed? It would be way better than it
is now, and errors during extension creation are rare and mostly for
developers only, so it doesn't have to be pretty.
> I'm inclined to think that maybe we'd be best off keeping the server
> end of it straightforward, and trying to teach psql to abbreviate the
> QUERY field in a useful way. IIRC you get this same type of problem
> with very large SQL-language functions and suchlike.
I'd treat this as a separate patch, if it's considered to be a good
idea.
Christoph
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: myon@debian.org, tgl@sss.pgh.pa.us, mbanck@gmx.net, tommypav122@gmail.com, ilmari@ilmari.org, pgsql-hackers@lists.postgresql.org, jian.universality@gmail.com
Subject: Re: Better error reporting from extension scripts (Was: Extend ALTER OPERATOR)
In-Reply-To: <ZvaiS57G10dITjQm@msg.df7cb.de>
* 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