Re: New version of money type

From: "D'Arcy J(dot)M(dot) Cain" <darcy(at)druid(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: jim(at)nasby(dot)net, kleptog(at)svana(dot)org, llonergan(at)greenplum(dot)com, sfrost(at)snowman(dot)net, pgsql-hackers(at)postgresql(dot)org
Subject: Re: New version of money type
Date: 2006-10-12 17:55:26
Message-ID: 20061012135526.2c666403.darcy@druid.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, 12 Oct 2006 13:21:37 -0400
Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > Well, the patch I submitted is definitely an improvement over the
> > existing version. Are you saying that I have to make further
> > improvements before these ones can be imported?
>
> I didn't say that. I was responding to someone whose position seemed to
> be "money is going to be removed, therefore you shouldn't work on it".
> I wanted to know exactly what would need to be fixed before they'd not
> want it removed.

Cool. So what do I do with the patch? Should I add the currency
symbol back in and commit or should I resubmit the patch to hackers for
further review?

--
D'Arcy J.M. Cain <darcy(at)druid(dot)net> | Democracy is three wolves
http://www.druid.net/darcy/ | and a sheep voting on
+1 416 425 1212 (DoD#0082) (eNTP) | what's for dinner.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2006-10-12 18:02:22 Re: array_accum aggregate
Previous Message Andrew Sullivan 2006-10-12 17:53:27 Re: On status data and summaries