| From: | Neil Conway <neilc(at)samurai(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
| Subject: | Re: reorder GROUP BY list |
| Date: | 2006-03-05 05:04:34 |
| Message-ID: | 1141535074.24513.64.camel@localhost.localdomain |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-patches |
On Sat, 2006-03-04 at 21:21 -0500, Tom Lane wrote:
> 1. For each ORDER BY item:
> 1a. Find a matching GROUP BY item.
> (Break out of loop if no match.)
> 1b. Add it to the output list with ORDER BY item's ordering op.
> 1c. Remove the matched item from the GROUP BY list.
> 2. For each remaining GROUP BY item:
> 2a. Add it to output list using default ordering op.
Okay, attached is a revised patch that implements this. Barring any
objections I'll apply it tomorrow.
-Neil
| Attachment | Content-Type | Size |
|---|---|---|
| group_by_reorder-5.patch | text/x-patch | 6.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2006-03-05 05:07:15 | Re: TODO-Item: include for guc |
| Previous Message | Sergey E. Koposov | 2006-03-05 03:57:19 | fix of some issues with multi-line query editing |