Robert Haas <robertmh...@gmail.com> writes: > Yeah, that looks like a good spot. I think we should also add > something to the documentation of the factorial operator, mentioning > that it will be going away. Perhaps we can advise people to write !!3 > instead of 3! for forward-compatibility, or maybe we should instead > suggest numeric_fac(3).
Well, the !! operator itself has been "deprecated" for a long time: regression=# \do+ !! List of operators Schema | Name | Left arg type | Right arg type | Result type | Function | Description ------------+------+---------------+----------------+-------------+-------------+--------------------------- pg_catalog | !! | | bigint | numeric | numeric_fac | deprecated, use ! instead pg_catalog | !! | | tsquery | tsquery | tsquery_not | NOT tsquery (2 rows) I'm a bit inclined to kill them both off and standardize on factorial() (not numeric_fac). regards, tom lane