diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2022-12-10 18:26:43 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2022-12-10 18:26:43 -0500 |
| commit | d02ef65bce6575e10ac0b4e7b8552ff67687c944 (patch) | |
| tree | 3a993d5826fea7932e4d127ddd9168327b495a72 /src/backend/utils/adt/tsgistidx.c | |
| parent | e730718072bdc8379f3d099c64455c20d6a6d1ad (diff) | |
| download | postgresql-d02ef65bce6575e10ac0b4e7b8552ff67687c944.tar.gz postgresql-d02ef65bce6575e10ac0b4e7b8552ff67687c944.zip | |
Standardize error reports in unimplemented I/O functions.
We chose a specific wording of the not-implemented errors for
pseudotype I/O functions and other cases where there's little
value in implementing input and/or output. gtsvectorin never
got that memo though, nor did most of contrib. Make these all
fall in line, mostly because I'm a neatnik but also to remove
unnecessary translatable strings.
gbtreekey_in needs a bit of extra love since it supports
multiple SQL types. Sadly, gbtreekey_out doesn't have the
ability to do that, but I think it's unreachable anyway.
Noted while surveying datatype input functions to see what we
have left to fix.
Diffstat (limited to 'src/backend/utils/adt/tsgistidx.c')
| -rw-r--r-- | src/backend/utils/adt/tsgistidx.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/backend/utils/adt/tsgistidx.c b/src/backend/utils/adt/tsgistidx.c index 728b5e9e713..cabfe4d1485 100644 --- a/src/backend/utils/adt/tsgistidx.c +++ b/src/backend/utils/adt/tsgistidx.c @@ -87,10 +87,12 @@ static int32 sizebitvec(BITVECP sign, int siglen); Datum gtsvectorin(PG_FUNCTION_ARGS) { + /* There's no need to support input of gtsvectors */ ereport(ERROR, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), - errmsg("gtsvector_in not implemented"))); - PG_RETURN_DATUM(0); + errmsg("cannot accept a value of type %s", "gtsvector"))); + + PG_RETURN_VOID(); /* keep compiler quiet */ } #define SINGOUTSTR "%d true bits, %d false bits" |