Thread: to_char() and negative intervals
Someone on IRC just reported this to_char() failure with negative
intervals:test=> SELECT to_char('-00:17:56.886375'::interval, 'HH24:MI:SS'); to_char---------- 00:-1:-5(1 row)
Looks like a bug to me. :-)
-- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610)
359-1001+ If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square,
Pennsylvania19073
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Someone on IRC just reported this to_char() failure with negative
> intervals:
I think Karel wanted to remove to_char(interval) altogether.
regards, tom lane
On Fri, Aug 13, 2004 at 12:24:28PM -0400, Tom Lane wrote: > Bruce Momjian <pgman@candle.pha.pa.us> writes: > > Someone on IRC just reported this to_char() failure with negative > > intervals: > > I think Karel wanted to remove to_char(interval) altogether. Yes, the notice about it is already in 7.4 docs. Maybe we can removeit in 7.5/8.0. Comments? Karel -- Karel Zak <zakkr@zf.jcu.cz>http://home.zf.jcu.cz/~zakkr/
Karel Zak wrote: > On Fri, Aug 13, 2004 at 12:24:28PM -0400, Tom Lane wrote: > > Bruce Momjian <pgman@candle.pha.pa.us> writes: > > > Someone on IRC just reported this to_char() failure with negative > > > intervals: > > > > I think Karel wanted to remove to_char(interval) altogether. > > Yes, the notice about it is already in 7.4 docs. Maybe we can remove > it in 7.5/8.0. Comments? I don't see how we can remove it in 8.0 unless we require an initdb. What we could do is to throw an error/warning in the code and remove it completely in 8.1. -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001+ If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania19073
On Mon, Aug 16, 2004 at 10:43:10AM -0400, Bruce Momjian wrote: > Karel Zak wrote: > > On Fri, Aug 13, 2004 at 12:24:28PM -0400, Tom Lane wrote: > > > Bruce Momjian <pgman@candle.pha.pa.us> writes: > > > > Someone on IRC just reported this to_char() failure with negative > > > > intervals: > > > > > > I think Karel wanted to remove to_char(interval) altogether. > > > > Yes, the notice about it is already in 7.4 docs. Maybe we can remove > > it in 7.5/8.0. Comments? > > I don't see how we can remove it in 8.0 unless we require an initdb. > What we could do is to throw an error/warning in the code and remove it > completely in 8.1. If we force an initdb for some other reason during beta, we can remove to_char(interval) at that point too ... -- Alvaro Herrera (<alvherre[a]dcc.uchile.cl>) "Siempre hay que alimentar a los dioses, aunque la tierra esté seca" (Orual)
Added to open items: * remove to_char(interval) if we initdb --------------------------------------------------------------------------- Alvaro Herrera wrote: > On Mon, Aug 16, 2004 at 10:43:10AM -0400, Bruce Momjian wrote: > > Karel Zak wrote: > > > On Fri, Aug 13, 2004 at 12:24:28PM -0400, Tom Lane wrote: > > > > Bruce Momjian <pgman@candle.pha.pa.us> writes: > > > > > Someone on IRC just reported this to_char() failure with negative > > > > > intervals: > > > > > > > > I think Karel wanted to remove to_char(interval) altogether. > > > > > > Yes, the notice about it is already in 7.4 docs. Maybe we can remove > > > it in 7.5/8.0. Comments? > > > > I don't see how we can remove it in 8.0 unless we require an initdb. > > What we could do is to throw an error/warning in the code and remove it > > completely in 8.1. > > If we force an initdb for some other reason during beta, we can remove > to_char(interval) at that point too ... > > -- > Alvaro Herrera (<alvherre[a]dcc.uchile.cl>) > "Siempre hay que alimentar a los dioses, aunque la tierra est? seca" (Orual) > > > ---------------------------(end of broadcast)--------------------------- > TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org > -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001+ If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania19073
We will either remove it in 8.0 or mention its removal in 8.1. It is probably wisest to remove in 8.1. --------------------------------------------------------------------------- Bruce Momjian wrote: > > Added to open items: > > * remove to_char(interval) if we initdb > > > --------------------------------------------------------------------------- > > Alvaro Herrera wrote: > > On Mon, Aug 16, 2004 at 10:43:10AM -0400, Bruce Momjian wrote: > > > Karel Zak wrote: > > > > On Fri, Aug 13, 2004 at 12:24:28PM -0400, Tom Lane wrote: > > > > > Bruce Momjian <pgman@candle.pha.pa.us> writes: > > > > > > Someone on IRC just reported this to_char() failure with negative > > > > > > intervals: > > > > > > > > > > I think Karel wanted to remove to_char(interval) altogether. > > > > > > > > Yes, the notice about it is already in 7.4 docs. Maybe we can remove > > > > it in 7.5/8.0. Comments? > > > > > > I don't see how we can remove it in 8.0 unless we require an initdb. > > > What we could do is to throw an error/warning in the code and remove it > > > completely in 8.1. > > > > If we force an initdb for some other reason during beta, we can remove > > to_char(interval) at that point too ... > > > > -- > > Alvaro Herrera (<alvherre[a]dcc.uchile.cl>) > > "Siempre hay que alimentar a los dioses, aunque la tierra est? seca" (Orual) > > > > > > ---------------------------(end of broadcast)--------------------------- > > TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org > > > > -- > Bruce Momjian | http://candle.pha.pa.us > pgman@candle.pha.pa.us | (610) 359-1001 > + If your life is a hard drive, | 13 Roberts Road > + Christ can be your backup. | Newtown Square, Pennsylvania 19073 > > ---------------------------(end of broadcast)--------------------------- > TIP 2: you can get off all lists at once with the unregister command > (send "unregister YourEmailAddressHere" to majordomo@postgresql.org) > -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001+ If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania19073