|From:||Andres Freund <andres(at)anarazel(dot)de>|
|To:||Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>|
|Subject:||Re: Implement targetlist SRFs using ROWS FROM() (was Changed SRF in targetlist handling)|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On 2016-09-12 16:56:32 -0700, Andres Freund wrote:
> On 2016-09-12 09:14:47 -0700, Andres Freund wrote:
> > On 2016-09-12 10:19:14 -0400, Tom Lane wrote:
> > > Andres Freund <andres(at)anarazel(dot)de> writes:
> > >
> > > > 0001-Add-some-more-targetlist-srf-tests.patch
> > > > Add some test.
> > >
> > > I think you should go ahead and push this tests-adding patch now, as it
> > > adds documentation of the current behavior that is a good thing to have
> > > independently of what the rest of the patchset does. Also, I'm worried
> > > that some of the GROUP BY tests might have machine-dependent results
> > > (if they are implemented by hashing) so it would be good to get in a few
> > > buildfarm cycles and let that settle out before we start changing the
> > > answers.
> > Generally a sound plan - I started to noticeably expand it though,
> > there's some important edge cases it didn't cover.
> Attached is a noticeably expanded set of tests, with fixes for the stuff
> you'd found. I plan to push that soon-ish. Independent of the approach
> we'll be choosing, increased coverage seems quite useful.
And for real.
|Next Message||Vitaly Burovoy||2016-09-12 23:59:39||Re: identity columns|
|Previous Message||Andres Freund||2016-09-12 23:56:32||Re: Implement targetlist SRFs using ROWS FROM() (was Changed SRF in targetlist handling)|