| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | pgsql-hackers(at)postgresql(dot)org, Bruce Momjian <bruce(at)momjian(dot)us> |
| Cc: | PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>, Roman Kononov <kononov195-pgsql(at)yahoo(dot)com> |
| Subject: | Re: [BUGS] BUG #2846: inconsistent and confusing handling of |
| Date: | 2006-12-27 19:15:02 |
| Message-ID: | 19208.1167246902@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers pgsql-patches |
Bruce Momjian <bruce(at)momjian(dot)us> writes:
> I have made some more progress on this patch.
I'm not convinced that you're fixing things so much as doing your best
to destroy IEEE-compliant float arithmetic behavior.
I think what we should probably consider is removing CheckFloat4Val
and CheckFloat8Val altogether, and just letting the float arithmetic
have its head. Most modern hardware gets float arithmetic right per
spec, and we shouldn't be second-guessing it.
A slightly less radical proposal is to reject only the case where
isinf(result) and neither input isinf(); and perhaps likewise with
respect to NaNs.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Simon Riggs | 2006-12-27 19:24:46 | Re: Bitmap index thoughts |
| Previous Message | dakotali kasap | 2006-12-27 19:08:24 | Re: how to add my source file? |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alvaro Herrera | 2006-12-27 20:22:35 | Re: Allow the identifier length to be increased via a configure option |
| Previous Message | Bruce Momjian | 2006-12-27 18:44:31 | Re: [BUGS] BUG #2846: inconsistent and confusing handling of |