Re: [PATCHES] Fixes for MONEY type using locale

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: "D'Arcy J(dot)M(dot) Cain" <darcy(at)druid(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCHES] Fixes for MONEY type using locale
Date: 2007-11-26 23:32:41
Message-ID: 200711262332.lAQNWfP26961@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Bruce Momjian wrote:
> D'Arcy J.M. Cain wrote:
> > On Sat, 24 Nov 2007 11:27:38 -0500 (EST)
> > Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> > > I am confused about two other items with MONEY. First, why can't
> > > anything but a string be cast to this type?
> > >
> > > test=> select 871234872319489323::money;
> > > ERROR: cannot cast type bigint to money
> > > LINE 1: select 871234872319489323::money;
> > > ^
> > > test=> select 871234872::money;
> > > ERROR: cannot cast type integer to money
> > > LINE 1: select 871234872::money;
> > > ^
> > > test=> select 87123487231.3::money;
> > > ERROR: cannot cast type numeric to money
> > > LINE 1: select 87123487231.3::money;
> > > ^
> >
> > I agree. I wasn't the one that added the meta information.
>
> OK, so the big question then is if we are un-depricating this data type,
> does it have the behavior we want? (And the regression addition will be
> helpful too.)

Added to TODO list:

* Allow MONEY to be cast to/from other numeric data types

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon Riggs 2007-11-26 23:39:10 Re: 8.3devel slower than 8.2 under read-only load
Previous Message Tom Lane 2007-11-26 23:18:45 Re: 8.3devel slower than 8.2 under read-only load

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2007-11-27 02:13:46 Proposed patch for operator lookup caching
Previous Message Tom Lane 2007-11-26 22:34:03 Re: [GENERAL] Empty arrays with ARRAY[]