Joe Conway <mail@joeconway.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.
regards, tom lane
