diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2016-04-01 13:35:48 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2016-04-01 13:35:48 -0300 |
| commit | f402b9950120358d1870aacc10070e121d8a17de (patch) | |
| tree | 901a199dae99a96485e1ae6e9a9ba94073972f14 /src/backend/commands/operatorcmds.c | |
| parent | a067b50470cf7fda77dd28b03519f2483c2322bf (diff) | |
| download | postgresql-f402b9950120358d1870aacc10070e121d8a17de.tar.gz postgresql-f402b9950120358d1870aacc10070e121d8a17de.zip | |
Type names should not be quoted
Our actual convention, contrary to what I said in 59a2111b23f, is not to
quote type names, as evidenced by unquoted use of format_type_be()
result value in error messages. Remove quotes from recently tweaked
messages accordingly.
Per note from Tom Lane
Diffstat (limited to 'src/backend/commands/operatorcmds.c')
| -rw-r--r-- | src/backend/commands/operatorcmds.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/commands/operatorcmds.c b/src/backend/commands/operatorcmds.c index 6edbe85ef53..672f2eb212f 100644 --- a/src/backend/commands/operatorcmds.c +++ b/src/backend/commands/operatorcmds.c @@ -275,7 +275,7 @@ ValidateRestrictionEstimator(List *restrictionName) if (get_func_rettype(restrictionOid) != FLOAT8OID) ereport(ERROR, (errcode(ERRCODE_INVALID_OBJECT_DEFINITION), - errmsg("restriction estimator function %s must return type \"%s\"", + errmsg("restriction estimator function %s must return type %s", NameListToString(restrictionName), "float8"))); /* Require EXECUTE rights for the estimator */ @@ -321,7 +321,7 @@ ValidateJoinEstimator(List *joinName) if (get_func_rettype(joinOid) != FLOAT8OID) ereport(ERROR, (errcode(ERRCODE_INVALID_OBJECT_DEFINITION), - errmsg("join estimator function %s must return type \"%s\"", + errmsg("join estimator function %s must return type %s", NameListToString(joinName), "float8"))); /* Require EXECUTE rights for the estimator */ |