Tom Lane wrote:
> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > Darcy Buskermolen wrote:
> >>> * remove to_char(interval) if we initdb or mention removal
> >>
> >> I vote just to mention it's removal at this time,
>
> > Agreed. Done.
>
> While I don't care that much one way or the other --- what is the
> difference between this and the prior state? Karel already said
> in the 7.4 docs that to_char(interval) would be removed in the next
> release. Why would the people who ignored the warning last time
> believe it this time round?
>
> I think that 8.0 is a more appropriate release number in which to be
> taking backwards-compatibility hits than 8.1. So if we're gonna do
> it at all, I would vote for doing it now.
I don't see any mention of it being removed in the 7.4 release notes.
I see it in the SGML docs:
Warning: <literal><function>to_char</function>(<type>interval</type>, <type>text</type>)</literal> is deprecated
andshould not be used in newly-written code. It will be removed in the next version.
I suppose that is enough warning. Is it fair to remove things during
beta though, especially since it wasn't mentioned in the 7.4 release
notes but just the docs? Of course, if it is buggy, that might be a
reason to remove it anyway.
Let's keep it in the release notes and if we remove it via initdb we can
just update the release notes to say it is now removed in 8.0.
-- 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
