> On Aug 27, 2020, at 7:04 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Robert Haas <robertmhaas@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
Just for historical context, it seems that when you committed 908ab80286401bb20a519fa7dc7a837631f20369 in 2011, you
werechoosing one operator per underlying proc to be the canonical operator name, and deprecating all other operators
basedon the same proc. You chose postfix ! as the canonical operator for numeric_fac and deprecated prefix !!, but I
thinkI can infer from that commit that if postfix ! did not exist, prefix !! would have been the canonical operator and
wouldnot have been deprecated.
The main reason I did not remove prefix !! in this patch series is that the patch is about removing postfix operator
support,and so it seemed off topic. But if there is general agreement to remove prefix !!, I'll put that in the next
patch.
—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
