Numeric 508 datatype

From: Simon Riggs <simon(at)2ndquadrant(dot)com>
To: pgsql-patches(at)postgresql(dot)org
Subject: Numeric 508 datatype
Date: 2005-11-16 17:13:49
Message-ID: 1132161229.4959.12.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general pgsql-hackers pgsql-patches


Now we're into 8.2devel mode, its time to submit the previously
discussed patch that:

- reduces Numeric storage format by 2 bytes
- limits scale to +/- 508 decimal places

This is sufficient to allow Numeric to continue to be used as the
default numeric representation for all numbers in the parser.

Passes: make check on cvstip, as well as some tests not in there.

Code comments explain the new format and consequences.

As previously agreed, reviewing this is a 2 stage process:
1. review/possibly agree OK to commit
2. check with everybody on GENERAL that the restriction to 508 is
acceptable

Figure there's no point doing (2) until we agree the proposal/code is
workable.

As Atsushi-san point out, there is also come CPU optimization to be done
on Numeric comparison, and also on other areas such as aggregation. I've
not done this yet.

Best Regards, Simon Riggs

Attachment Content-Type Size
num508.patch text/x-patch 21.6 KB

Responses

Browse pgsql-general by date

  From Date Subject
Next Message Tom Lane 2005-11-16 17:20:26 Re: Rebranding PostgreSQL
Previous Message Scott Marlowe 2005-11-16 17:12:42 Re: Rebranding PostgreSQL

Browse pgsql-hackers by date

  From Date Subject
Next Message Jim C. Nasby 2005-11-16 17:15:07 Re: OS X 7.4 failure
Previous Message Tom Lane 2005-11-16 17:12:12 Re: MERGE vs REPLACE

Browse pgsql-patches by date

  From Date Subject
Next Message Philip Yarra 2005-11-16 22:26:55 Re: [PATCHES] pl/pgSQL doco patch
Previous Message Bruce Momjian 2005-11-16 16:38:14 Re: drop if exists