pgsql: Downgrade implicit casts to text to be assignment-only, except

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Downgrade implicit casts to text to be assignment-only, except
Date: 2007-06-05 21:31:09
Message-ID: 20070605213109.B61E19FB78F@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Downgrade implicit casts to text to be assignment-only, except for the ones
from the other string-category types; this eliminates a lot of surprising
interpretations that the parser could formerly make when there was no directly
applicable operator.

Create a general mechanism that supports casts to and from the standard string
types (text,varchar,bpchar) for *every* datatype, by invoking the datatype's
I/O functions. These new casts are assignment-only in the to-string direction,
explicit-only in the other, and therefore should create no surprising behavior.
Remove a bunch of thereby-obsoleted datatype-specific casting functions.

The "general mechanism" is a new expression node type CoerceViaIO that can
actually convert between *any* two datatypes if their external text
representations are compatible. This is more general than needed for the
immediate feature, but might be useful in plpgsql or other places in future.

This commit does nothing about the issue that applying the concatenation
operator || to non-text types will now fail, often with strange error messages
due to misinterpreting the operator as array concatenation. Since it often
(not always) worked before, we should either make it succeed or at least give
a more user-friendly error; but details are still under debate.

Peter Eisentraut and Tom Lane

Modified Files:
--------------
pgsql/contrib/cube:
README.cube (r1.9 -> r1.10)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/cube/README.cube.diff?r1=1.9&r2=1.10)
cube.c (r1.32 -> r1.33)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/cube/cube.c.diff?r1=1.32&r2=1.33)
cube.sql.in (r1.19 -> r1.20)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/cube/cube.sql.in.diff?r1=1.19&r2=1.20)
uninstall_cube.sql (r1.4 -> r1.5)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/cube/uninstall_cube.sql.diff?r1=1.4&r2=1.5)
pgsql/contrib/cube/expected:
cube.out (r1.20 -> r1.21)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/cube/expected/cube.out.diff?r1=1.20&r2=1.21)
cube_1.out (r1.8 -> r1.9)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/cube/expected/cube_1.out.diff?r1=1.8&r2=1.9)
cube_2.out (r1.5 -> r1.6)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/cube/expected/cube_2.out.diff?r1=1.5&r2=1.6)
pgsql/contrib/cube/sql:
cube.sql (r1.11 -> r1.12)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/cube/sql/cube.sql.diff?r1=1.11&r2=1.12)
pgsql/contrib/isn:
isn.c (r1.5 -> r1.6)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/isn/isn.c.diff?r1=1.5&r2=1.6)
isn.h (r1.3 -> r1.4)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/isn/isn.h.diff?r1=1.3&r2=1.4)
isn.sql.in (r1.4 -> r1.5)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/isn/isn.sql.in.diff?r1=1.4&r2=1.5)
pgsql/doc/src/sgml:
catalogs.sgml (r2.152 -> r2.153)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/catalogs.sgml.diff?r1=2.152&r2=2.153)
syntax.sgml (r1.118 -> r1.119)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/syntax.sgml.diff?r1=1.118&r2=1.119)
typeconv.sgml (r1.51 -> r1.52)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/typeconv.sgml.diff?r1=1.51&r2=1.52)
pgsql/doc/src/sgml/ref:
create_cast.sgml (r1.25 -> r1.26)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/create_cast.sgml.diff?r1=1.25&r2=1.26)
pgsql/src/backend/catalog:
dependency.c (r1.65 -> r1.66)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/catalog/dependency.c.diff?r1=1.65&r2=1.66)
pgsql/src/backend/executor:
execQual.c (r1.217 -> r1.218)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execQual.c.diff?r1=1.217&r2=1.218)
pgsql/src/backend/nodes:
copyfuncs.c (r1.376 -> r1.377)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/copyfuncs.c.diff?r1=1.376&r2=1.377)
equalfuncs.c (r1.307 -> r1.308)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/equalfuncs.c.diff?r1=1.307&r2=1.308)
outfuncs.c (r1.308 -> r1.309)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/outfuncs.c.diff?r1=1.308&r2=1.309)
readfuncs.c (r1.206 -> r1.207)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/readfuncs.c.diff?r1=1.206&r2=1.207)
pgsql/src/backend/optimizer/path:
costsize.c (r1.183 -> r1.184)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/costsize.c.diff?r1=1.183&r2=1.184)
pgsql/src/backend/optimizer/util:
clauses.c (r1.244 -> r1.245)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/util/clauses.c.diff?r1=1.244&r2=1.245)
pgsql/src/backend/parser:
parse_coerce.c (r2.153 -> r2.154)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/parse_coerce.c.diff?r1=2.153&r2=2.154)
parse_expr.c (r1.217 -> r1.218)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/parse_expr.c.diff?r1=1.217&r2=1.218)
parse_func.c (r1.195 -> r1.196)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/parse_func.c.diff?r1=1.195&r2=1.196)
pgsql/src/backend/utils:
Gen_fmgrtab.sh (r1.35 -> r1.36)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/Gen_fmgrtab.sh.diff?r1=1.35&r2=1.36)
pgsql/src/backend/utils/adt:
bool.c (r1.39 -> r1.40)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/bool.c.diff?r1=1.39&r2=1.40)
date.c (r1.131 -> r1.132)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/date.c.diff?r1=1.131&r2=1.132)
enum.c (r1.2 -> r1.3)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/enum.c.diff?r1=1.2&r2=1.3)
float.c (r1.149 -> r1.150)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/float.c.diff?r1=1.149&r2=1.150)
int.c (r1.79 -> r1.80)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/int.c.diff?r1=1.79&r2=1.80)
int8.c (r1.65 -> r1.66)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/int8.c.diff?r1=1.65&r2=1.66)
mac.c (r1.37 -> r1.38)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/mac.c.diff?r1=1.37&r2=1.38)
network.c (r1.70 -> r1.71)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/network.c.diff?r1=1.70&r2=1.71)
numeric.c (r1.102 -> r1.103)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/numeric.c.diff?r1=1.102&r2=1.103)
oid.c (r1.71 -> r1.72)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/oid.c.diff?r1=1.71&r2=1.72)
regproc.c (r1.100 -> r1.101)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/regproc.c.diff?r1=1.100&r2=1.101)
ri_triggers.c (r1.94 -> r1.95)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/ri_triggers.c.diff?r1=1.94&r2=1.95)
ruleutils.c (r1.258 -> r1.259)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/ruleutils.c.diff?r1=1.258&r2=1.259)
timestamp.c (r1.176 -> r1.177)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/timestamp.c.diff?r1=1.176&r2=1.177)
uuid.c (r1.3 -> r1.4)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/uuid.c.diff?r1=1.3&r2=1.4)
pgsql/src/include/catalog:
catversion.h (r1.409 -> r1.410)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/catversion.h.diff?r1=1.409&r2=1.410)
pg_cast.h (r1.33 -> r1.34)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/pg_cast.h.diff?r1=1.33&r2=1.34)
pg_proc.h (r1.457 -> r1.458)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/pg_proc.h.diff?r1=1.457&r2=1.458)
pgsql/src/include/nodes:
execnodes.h (r1.175 -> r1.176)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/execnodes.h.diff?r1=1.175&r2=1.176)
nodes.h (r1.199 -> r1.200)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/nodes.h.diff?r1=1.199&r2=1.200)
primnodes.h (r1.129 -> r1.130)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/primnodes.h.diff?r1=1.129&r2=1.130)
pgsql/src/include/parser:
parse_coerce.h (r1.70 -> r1.71)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/parser/parse_coerce.h.diff?r1=1.70&r2=1.71)
pgsql/src/include/utils:
builtins.h (r1.294 -> r1.295)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/builtins.h.diff?r1=1.294&r2=1.295)
date.h (r1.36 -> r1.37)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/date.h.diff?r1=1.36&r2=1.37)
int8.h (r1.46 -> r1.47)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/int8.h.diff?r1=1.46&r2=1.47)
timestamp.h (r1.69 -> r1.70)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/timestamp.h.diff?r1=1.69&r2=1.70)
pgsql/src/pl/plpgsql/src:
pl_exec.c (r1.196 -> r1.197)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/pl/plpgsql/src/pl_exec.c.diff?r1=1.196&r2=1.197)
pgsql/src/test/regress/expected:
foreign_key.out (r1.42 -> r1.43)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/foreign_key.out.diff?r1=1.42&r2=1.43)
strings.out (r1.31 -> r1.32)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/strings.out.diff?r1=1.31&r2=1.32)
subselect.out (r1.14 -> r1.15)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/subselect.out.diff?r1=1.14&r2=1.15)
pgsql/src/test/regress/sql:
foreign_key.sql (r1.18 -> r1.19)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/foreign_key.sql.diff?r1=1.18&r2=1.19)
strings.sql (r1.20 -> r1.21)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/strings.sql.diff?r1=1.20&r2=1.21)
subselect.sql (r1.9 -> r1.10)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/subselect.sql.diff?r1=1.9&r2=1.10)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2007-06-05 21:50:19 pgsql: Minor editorialization: don't flush plan cache without need.
Previous Message Jan Wieck 2007-06-05 20:00:42 pgsql: The session_replication_role actually can be changed at will