From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] [COMMITTERS] pgsql: Change FETCH/MOVE |
Date: | 2006-09-03 03:24:40 |
Message-ID: | 200609030324.k833Oet24439@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers pgsql-patches |
Tom Lane wrote:
> Bruce Momjian <bruce(at)momjian(dot)us> writes:
> >> This patch has broken half the buildfarm, and I've still not seen a
> >> rationale why we need to make such a change at all.
>
> > Fixed with attached patch. The use case for this was not FETCH, but
> > MOVE for > 2gig tables.
>
> There is *no* credible use case for this (hint: MOVE FORWARD/BACKWARD
> ALL does not need this to work for >2G tables). It is not worth the
> extra computational cycles that it imposes on every machine whether they
> use the feature or not, and it is certainly not worth the developer time
> we're expending to fix this poorly written patch. Please revert it.
Already done because of bad coding. You want the TODO item removed too?
--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2006-09-03 03:26:42 | pgsql: Reverted: < * -Change LIMIT/OFFSET and FETCH/MOVE to use int8 > |
Previous Message | Tom Lane | 2006-09-03 03:22:35 | Re: [COMMITTERS] pgsql: Change FETCH/MOVE to use int8. |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2006-09-03 03:39:33 | Re: plpgsql, return can contains any expression |
Previous Message | Tom Lane | 2006-09-03 03:22:35 | Re: [COMMITTERS] pgsql: Change FETCH/MOVE to use int8. |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2006-09-03 03:26:06 | Re: Patch for - Change FETCH/MOVE to use int8 |
Previous Message | Tom Lane | 2006-09-03 03:22:35 | Re: [COMMITTERS] pgsql: Change FETCH/MOVE to use int8. |