pg.ddx.io pgsql-hackers@postgresql.org mailing list archive
help / color / mirror / Atom feedSome doubious error messages and comments
4+ messages / 3 participants
[nested] [flat]
* Some doubious error messages and comments
@ 2021-04-28 08:36 Kyotaro Horiguchi <horikyota.ntt@gmail.com>
2021-04-28 13:11 ` Re: Some doubious error messages and comments Justin Pryzby <pryzby@telsasoft.com>
0 siblings, 1 reply; 4+ messages in thread
From: Kyotaro Horiguchi @ 2021-04-28 08:36 UTC (permalink / raw)
To: pgsql-hackers@lists.postgresql.org
Hello.
0001: I found some typos in a error message and a comment.
multirangetypes.c: 1420
> errmsg("range_intersect_agg must be called with a multirange")));
This "range_intersect_agg" looks like a typo of "multirange_..".
operatorcmds.c:303
> * Look up a join estimator function ny name, and verify that it has the
"ny" looks like a typo of "by".
0002: The following messages are substantially same and are uselessly
split into separate messages. I'm not sure any compiler complains
about using %zu for int, explicit casting would work in that case.
be-secure-gssapi.c:351
> (errmsg("oversize GSSAPI packet sent by the client (%zu > %zu)",
> (size_t) input.length,
> PQ_GSS_RECV_BUFFER_SIZE - sizeof(uint32))));
be-secure-gssapi.c:570
> (errmsg("oversize GSSAPI packet sent by the client (%zu > %d)",
> (size_t) input.length,
> PQ_GSS_RECV_BUFFER_SIZE)));
0003: The messages below seems to be a bit unclear. I'm not sure they
worth doing.
conversioncmds.c: 130
errmsg("encoding conversion function %s returned incorrect result for empty input",
This is not wrong at all, but another message just above is saying
that "encoding conversion function %s must return type %s". Why
aren't we explicit here, like this?
"encoding conversion function %s must return zero for empty input"
typecmds.c:4294
> if (requireSuper)
> if (!superuser())
> ereport(ERROR,
> errmsg("must be superuser to alter a type")));
Where, requireSuper varies depending on the set of operations but the
description looks like describing general behavior. I'm not sure but
something like the following might be better?
+ errmsg("must be superuser to perform all operations")));
+ errmsg("some of the operations require superuser privilege")));
Any opinions or suggestions?
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
Attachments:
[text/x-patch] 0001-Fix-typos.patch (1.5K, ../../20210428.173633.1525059946206239295.horikyota.ntt@gmail.com/2-0001-Fix-typos.patch)
download | inline diff:
From 1634d8dd87b1474f60cfd6689c1e58acc87c1cfc Mon Sep 17 00:00:00 2001
From: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Date: Wed, 28 Apr 2021 17:23:52 +0900
Subject: [PATCH 1/3] Fix typos
---
src/backend/commands/operatorcmds.c | 2 +-
src/backend/utils/adt/multirangetypes.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/backend/commands/operatorcmds.c b/src/backend/commands/operatorcmds.c
index 809043c5d1..ba456c4dd1 100644
--- a/src/backend/commands/operatorcmds.c
+++ b/src/backend/commands/operatorcmds.c
@@ -300,7 +300,7 @@ ValidateRestrictionEstimator(List *restrictionName)
}
/*
- * Look up a join estimator function ny name, and verify that it has the
+ * Look up a join estimator function by name, and verify that it has the
* correct signature and we have the permissions to attach it to an
* operator.
*/
diff --git a/src/backend/utils/adt/multirangetypes.c b/src/backend/utils/adt/multirangetypes.c
index 0b81649779..2583ddeedf 100644
--- a/src/backend/utils/adt/multirangetypes.c
+++ b/src/backend/utils/adt/multirangetypes.c
@@ -1417,7 +1417,7 @@ multirange_intersect_agg_transfn(PG_FUNCTION_ARGS)
if (!type_is_multirange(mltrngtypoid))
ereport(ERROR,
(errcode(ERRCODE_DATATYPE_MISMATCH),
- errmsg("range_intersect_agg must be called with a multirange")));
+ errmsg("multirange_intersect_agg must be called with a multirange")));
typcache = multirange_get_typcache(fcinfo, mltrngtypoid);
--
2.27.0
[text/x-patch] 0002-Consolidate-substantially-same-messages.patch (944B, ../../20210428.173633.1525059946206239295.horikyota.ntt@gmail.com/3-0002-Consolidate-substantially-same-messages.patch)
download | inline diff:
From 14a6b4b81d3b4d35bf690e1c240975bd888531f3 Mon Sep 17 00:00:00 2001
From: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Date: Wed, 28 Apr 2021 17:24:31 +0900
Subject: [PATCH 2/3] Consolidate substantially same messages
---
src/backend/libpq/be-secure-gssapi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/backend/libpq/be-secure-gssapi.c b/src/backend/libpq/be-secure-gssapi.c
index 316ca65db5..a335f78d47 100644
--- a/src/backend/libpq/be-secure-gssapi.c
+++ b/src/backend/libpq/be-secure-gssapi.c
@@ -567,9 +567,9 @@ secure_open_gssapi(Port *port)
if (input.length > PQ_GSS_RECV_BUFFER_SIZE)
{
ereport(COMMERROR,
- (errmsg("oversize GSSAPI packet sent by the client (%zu > %d)",
+ (errmsg("oversize GSSAPI packet sent by the client (%zu > %zu)",
(size_t) input.length,
- PQ_GSS_RECV_BUFFER_SIZE)));
+ (size_t) PQ_GSS_RECV_BUFFER_SIZE)));
return -1;
}
--
2.27.0
[text/x-patch] 0003-Clarify-merror-messages.patch (1.4K, ../../20210428.173633.1525059946206239295.horikyota.ntt@gmail.com/4-0003-Clarify-merror-messages.patch)
download | inline diff:
From 0db041334e0fc4df46f1d98e75e984b04777befe Mon Sep 17 00:00:00 2001
From: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Date: Wed, 28 Apr 2021 17:25:36 +0900
Subject: [PATCH 3/3] Clarify merror messages
---
src/backend/commands/conversioncmds.c | 2 +-
src/backend/commands/typecmds.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/backend/commands/conversioncmds.c b/src/backend/commands/conversioncmds.c
index 5fed97a2f9..307afb909d 100644
--- a/src/backend/commands/conversioncmds.c
+++ b/src/backend/commands/conversioncmds.c
@@ -127,7 +127,7 @@ CreateConversionCommand(CreateConversionStmt *stmt)
if (DatumGetInt32(funcresult) != 0)
ereport(ERROR,
(errcode(ERRCODE_INVALID_OBJECT_DEFINITION),
- errmsg("encoding conversion function %s returned incorrect result for empty input",
+ errmsg("encoding conversion function %s must return zero for empty input",
NameListToString(func_name))));
/*
diff --git a/src/backend/commands/typecmds.c b/src/backend/commands/typecmds.c
index 036fa69d17..3dd4323386 100644
--- a/src/backend/commands/typecmds.c
+++ b/src/backend/commands/typecmds.c
@@ -4291,7 +4291,7 @@ AlterType(AlterTypeStmt *stmt)
if (!superuser())
ereport(ERROR,
(errcode(ERRCODE_INSUFFICIENT_PRIVILEGE),
- errmsg("must be superuser to alter a type")));
+ errmsg("must be superuser to perform all operations")));
}
else
{
--
2.27.0
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: Some doubious error messages and comments
2021-04-28 08:36 Some doubious error messages and comments Kyotaro Horiguchi <horikyota.ntt@gmail.com>
@ 2021-04-28 13:11 ` Justin Pryzby <pryzby@telsasoft.com>
2021-05-10 06:52 ` Re: Some doubious error messages and comments Michael Paquier <michael@paquier.xyz>
0 siblings, 1 reply; 4+ messages in thread
From: Justin Pryzby @ 2021-04-28 13:11 UTC (permalink / raw)
To: Kyotaro Horiguchi <horikyota.ntt@gmail.com>; +Cc: pgsql-hackers@lists.postgresql.org
On Wed, Apr 28, 2021 at 05:36:33PM +0900, Kyotaro Horiguchi wrote:
> 0001: I found some typos in a error message and a comment.
>
> multirangetypes.c: 1420
> > errmsg("range_intersect_agg must be called with a multirange")));
>
> This "range_intersect_agg" looks like a typo of "multirange_..".
>
> operatorcmds.c:303
> > * Look up a join estimator function ny name, and verify that it has the
>
> "ny" looks like a typo of "by".
"ny name" shows up a 2nd time.
I have another "comment typos" patch - maybe someone will want to push them
together.
commit 32e979c652c68ca5e3a7f308d677058e0c08547b
Author: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Date: Wed Apr 28 17:23:52 2021 +0900
Fix typos
ny name: 321eed5f0f7563a0cabb3d7a98132856287c1ad1
multirange: 6df7a9698bb036610c1e8c6d375e1be38cb26d5f
diff --git a/src/backend/commands/operatorcmds.c b/src/backend/commands/operatorcmds.c
index 809043c5d1..fbd7d8d062 100644
--- a/src/backend/commands/operatorcmds.c
+++ b/src/backend/commands/operatorcmds.c
@@ -265,7 +265,7 @@ DefineOperator(List *names, List *parameters)
}
/*
- * Look up a restriction estimator function ny name, and verify that it has
+ * Look up a restriction estimator function by name, and verify that it has
* the correct signature and we have the permissions to attach it to an
* operator.
*/
@@ -300,7 +300,7 @@ ValidateRestrictionEstimator(List *restrictionName)
}
/*
- * Look up a join estimator function ny name, and verify that it has the
+ * Look up a join estimator function by name, and verify that it has the
* correct signature and we have the permissions to attach it to an
* operator.
*/
diff --git a/src/backend/utils/adt/multirangetypes.c b/src/backend/utils/adt/multirangetypes.c
index 0b81649779..2583ddeedf 100644
--- a/src/backend/utils/adt/multirangetypes.c
+++ b/src/backend/utils/adt/multirangetypes.c
@@ -1417,7 +1417,7 @@ multirange_intersect_agg_transfn(PG_FUNCTION_ARGS)
if (!type_is_multirange(mltrngtypoid))
ereport(ERROR,
(errcode(ERRCODE_DATATYPE_MISMATCH),
- errmsg("range_intersect_agg must be called with a multirange")));
+ errmsg("multirange_intersect_agg must be called with a multirange")));
typcache = multirange_get_typcache(fcinfo, mltrngtypoid);
commit 8247b4034ed4c68241be9fbdec249bc967ceafd4
Author: Justin Pryzby <pryzbyj@telsasoft.com>
Date: Tue Apr 27 07:57:50 2021 -0500
Comment typos: extended stats a4d75c86b and 518442c7f
diff --git a/src/backend/parser/parse_utilcmd.c b/src/backend/parser/parse_utilcmd.c
index 9dd30370da..eb9e63f4a8 100644
--- a/src/backend/parser/parse_utilcmd.c
+++ b/src/backend/parser/parse_utilcmd.c
@@ -1943,7 +1943,7 @@ generateClonedExtStatsStmt(RangeVar *heapRel, Oid heapRelid,
* simply append them after simple column references.
*
* XXX Some places during build/estimation treat expressions as if they
- * are before atttibutes, but for the CREATE command that's entirely
+ * are before attributes, but for the CREATE command that's entirely
* irrelevant.
*/
datum = SysCacheGetAttr(STATEXTOID, ht_stats,
diff --git a/src/backend/statistics/extended_stats.c b/src/backend/statistics/extended_stats.c
index 7e11cb9d5f..5e53783ea6 100644
--- a/src/backend/statistics/extended_stats.c
+++ b/src/backend/statistics/extended_stats.c
@@ -1796,7 +1796,7 @@ statext_mcv_clauselist_selectivity(PlannerInfo *root, List *clauses, int varReli
continue;
/*
- * Now we know the clause is compatible (we have either atttnums
+ * Now we know the clause is compatible (we have either attnums
* or expressions extracted from it), and was not estimated yet.
*/
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: Some doubious error messages and comments
2021-04-28 08:36 Some doubious error messages and comments Kyotaro Horiguchi <horikyota.ntt@gmail.com>
2021-04-28 13:11 ` Re: Some doubious error messages and comments Justin Pryzby <pryzby@telsasoft.com>
@ 2021-05-10 06:52 ` Michael Paquier <michael@paquier.xyz>
2021-05-14 06:06 ` Re: Some doubious error messages and comments Kyotaro Horiguchi <horikyota.ntt@gmail.com>
0 siblings, 1 reply; 4+ messages in thread
From: Michael Paquier @ 2021-05-10 06:52 UTC (permalink / raw)
To: Justin Pryzby <pryzby@telsasoft.com>; +Cc: Kyotaro Horiguchi <horikyota.ntt@gmail.com>; pgsql-hackers@lists.postgresql.org
On Wed, Apr 28, 2021 at 08:11:47AM -0500, Justin Pryzby wrote:
> On Wed, Apr 28, 2021 at 05:36:33PM +0900, Kyotaro Horiguchi wrote:
>> 0001: I found some typos in a error message and a comment.
>>
>> multirangetypes.c: 1420
>> > errmsg("range_intersect_agg must be called with a multirange")));
>>
>> This "range_intersect_agg" looks like a typo of "multirange_..".
>>
>> operatorcmds.c:303
>> > * Look up a join estimator function ny name, and verify that it has the
>>
>> "ny" looks like a typo of "by".
>
> "ny name" shows up a 2nd time.
>
> I have another "comment typos" patch - maybe someone will want to push them
> together.
Thanks. Two of them were already fixed, two of them are correct but
went missing so I have applied a fix for these. The change in
multirange_intersect_agg_transfn() is incorrect as the error refers to
the SQL function range_intersect_agg().
--
Michael
Attachments:
[application/pgp-signature] signature.asc (832B, ../../YJjYH6MeZ4tnjfSs@paquier.xyz/2-signature.asc)
download
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: Some doubious error messages and comments
2021-04-28 08:36 Some doubious error messages and comments Kyotaro Horiguchi <horikyota.ntt@gmail.com>
2021-04-28 13:11 ` Re: Some doubious error messages and comments Justin Pryzby <pryzby@telsasoft.com>
2021-05-10 06:52 ` Re: Some doubious error messages and comments Michael Paquier <michael@paquier.xyz>
@ 2021-05-14 06:06 ` Kyotaro Horiguchi <horikyota.ntt@gmail.com>
0 siblings, 0 replies; 4+ messages in thread
From: Kyotaro Horiguchi @ 2021-05-14 06:06 UTC (permalink / raw)
To: michael@paquier.xyz; +Cc: pryzby@telsasoft.com; pgsql-hackers@lists.postgresql.org
At Mon, 10 May 2021 15:52:15 +0900, Michael Paquier <michael@paquier.xyz> wrote in
> On Wed, Apr 28, 2021 at 08:11:47AM -0500, Justin Pryzby wrote:
> > On Wed, Apr 28, 2021 at 05:36:33PM +0900, Kyotaro Horiguchi wrote:
> >> 0001: I found some typos in a error message and a comment.
> >>
> >> multirangetypes.c: 1420
> >> > errmsg("range_intersect_agg must be called with a multirange")));
> >>
> >> This "range_intersect_agg" looks like a typo of "multirange_..".
> >>
> >> operatorcmds.c:303
> >> > * Look up a join estimator function ny name, and verify that it has the
> >>
> >> "ny" looks like a typo of "by".
> >
> > "ny name" shows up a 2nd time.
> >
> > I have another "comment typos" patch - maybe someone will want to push them
> > together.
>
> Thanks. Two of them were already fixed, two of them are correct but
> went missing so I have applied a fix for these. The change in
Thanks.
> multirange_intersect_agg_transfn() is incorrect as the error refers to
> the SQL function range_intersect_agg().
Uh!! It seems to be a never-shown message. Thaks for checking that.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
^ permalink raw reply [nested|flat] 4+ messages in thread
end of thread, other threads:[~2021-05-14 06:06 UTC | newest]
Thread overview: 4+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2021-04-28 08:36 Some doubious error messages and comments Kyotaro Horiguchi <horikyota.ntt@gmail.com>
2021-04-28 13:11 ` Justin Pryzby <pryzby@telsasoft.com>
2021-05-10 06:52 ` Michael Paquier <michael@paquier.xyz>
2021-05-14 06:06 ` Kyotaro Horiguchi <horikyota.ntt@gmail.com>
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