pgsql: Back out patch added during 8.2.X development: Allow to_char()

Started by Bruce Momjianover 19 years ago3 messageshackers
Jump to latest
#1Bruce Momjian
bruce@momjian.us

Log Message:
-----------
Back out patch added during 8.2.X development:

Allow to_char() "D" format specifiers for interval/time.

It doesn't work, and I doubt it is useful enough to fix ("D" = day of
week).

Modified Files:
--------------
pgsql/src/backend/utils/adt:
formatting.c (r1.111 -> r1.112)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/formatting.c.diff?r1=1.111&r2=1.112)

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bruce Momjian (#1)
Re: [COMMITTERS] pgsql: Back out patch added during 8.2.X development: Allow to_char()

momjian@postgresql.org (Bruce Momjian) writes:

Back out patch added during 8.2.X development:
Allow to_char() "D" format specifiers for interval/time.
It doesn't work, and I doubt it is useful enough to fix ("D" = day of
week).

Hm, shouldn't there be a documentation update to go with that? Or was
it never documented anyway :-(

regards, tom lane

#3Bruce Momjian
bruce@momjian.us
In reply to: Tom Lane (#2)
Re: [COMMITTERS] pgsql: Back out patch added during

Tom Lane wrote:

momjian@postgresql.org (Bruce Momjian) writes:

Back out patch added during 8.2.X development:
Allow to_char() "D" format specifiers for interval/time.
It doesn't work, and I doubt it is useful enough to fix ("D" = day of
week).

Hm, shouldn't there be a documentation update to go with that? Or was
it never documented anyway :-(

Not documented. Using "D" for an interval/time is more of a corner case
/ "don't do that". ;-)

--
Bruce Momjian bruce@momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +