Re: misleading error message in ProcessUtilitySlow T_CreateStatsStmt

Peter Eisentraut <peter@eisentraut.org>

From: Peter Eisentraut <peter@eisentraut.org>

To: Tom Lane <tgl@sss.pgh.pa.us>

Cc: Álvaro Herrera <alvherre@kurilemu.de>, jian he <jian.universality@gmail.com>, PostgreSQL-development <pgsql-hackers@postgresql.org>, Kirill Reshke <reshkekirill@gmail.com>

Date: 2025-09-15T10:16:39Z

Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

CREATE STATISTICS: improve misleading error message
- cbd3732cbbaa 14.20 landed
- 4529861cc408 15.15 landed
- f30c04682cdf 16.11 landed
- f9b41f3e1ecd 17.7 landed
- 38883916e41c 18.0 landed
- 9ec0b29976b6 19 (unreleased) landed
- f9a7622483c3 15.15 landed
- f225473cbae2 19 (unreleased) landed
- e9fb314c7ad8 17.7 landed
- d84a6c3dad1f 16.11 landed
- b830f9d67463 14.20 landed
- 3eea4dc2c7cf 18.0 landed
Glossary: improve definition of "relation"
- 16a9165ce4a4 19 (unreleased) landed
Extended statistics on expressions
- a4d75c86bf15 14.0 cited

On 12.09.25 15:49, Tom Lane wrote:
> Peter Eisentraut <peter@eisentraut.org> writes:
>> I propose the attached patch to fix this.  I think this restores the
>> original meaning better.
> 
> I'm okay with this wording change, but I would stay with
> ERRCODE_FEATURE_NOT_SUPPORTED rather than calling this
> a "syntax error".  It's not a syntax error IMV, but a
> potential feature that we have deliberately left syntax
> space for, even though we don't yet have ideas about
> a workable implementation.

Ok, done that way.