| From: | "Scott Marlowe" <scott(dot)marlowe(at)gmail(dot)com> |
|---|---|
| To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | "Joe Conway" <mail(at)joeconway(dot)com>, "Jorge Godoy" <jgodoy(at)gmail(dot)com>, pgsql-general(at)postgresql(dot)org, "Stefan Schwarzer" <stefan(dot)schwarzer(at)grid(dot)unep(dot)ch> |
| Subject: | Re: Crosstab Problems |
| Date: | 2007-10-25 14:29:33 |
| Message-ID: | dcc563d10710250729s45cc418apf4044003b077f0cb@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-general pgsql-patches |
On 10/24/07, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Joe Conway <mail(at)joeconway(dot)com> writes:
> > Tom Lane wrote:
> >> 1. Treat NULL rowid as a category in its own right. This would conform
> >> with the behavior of GROUP BY and DISTINCT, for instance.
>
> > In any case, the attached changes the behavior to #1 for both flavors of
> > crosstab (the original crosstab(text, int) and the usually more useful
> > crosstab(text, text)).
>
> > It is appropriate for 8.3 but not back-patching as it changes behavior
> > in a non-backward compatible way and is probably too invasive anyway.
>
> Um, if the previous code crashed in this case, why would you worry about
> being backward-compatible with it? You're effectively changing the
> behavior anyway, so you might as well make it do what you've decided is
> the right thing.
As a crosstab user, I agree with Tom.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2007-10-25 14:30:40 | Re: select count() out of memory |
| Previous Message | Roberts, Jon | 2007-10-25 14:25:13 | Re: subversion support? |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alvaro Herrera | 2007-10-25 14:45:55 | pgsql: Extract catalog info for error reporting before an error actually |
| Previous Message | Alvaro Herrera | 2007-10-25 12:29:25 | Re: two new chklocale aliases |