Re: Numeric version of factorial() - Mailing list pgsql-patches

From Tom Lane
Subject Re: Numeric version of factorial()
Date
Msg-id 3765.1059703681@sss.pgh.pa.us
Whole thread Raw
In response to Re: Numeric version of factorial()  (Gavin Sherry <swm@linuxworld.com.au>)
Responses Re: Numeric version of factorial()  (Gavin Sherry <swm@linuxworld.com.au>)
List pgsql-patches
Gavin Sherry <swm@linuxworld.com.au> writes:
> What are your feelings about numeric argument vs. int4/int8 arguments?

Actually I think it'd be fine to take int8.  We'd not be able to cope
with any larger input anyway, and the inner loop could be noticeably
faster if the control logic just deals with int.

We could leave the factorial(numeric) case open for a future
implementation that uses gamma, if anyone gets hot to do it.

            regards, tom lane

pgsql-patches by date:

Previous
From: Gavin Sherry
Date:
Subject: Re: Numeric version of factorial()
Next
From: "Serguei Mokhov"
Date:
Subject: Russian NLS Update: psql