| From: | Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Jeevan Chalke <jeevan(dot)chalke(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Strange result with LATERAL query |
| Date: | 2016-08-24 16:22:21 |
| Message-ID: | 87a8g2b0p6.fsf@news-spur.riddles.org.uk |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
>>>>> "Tom" == Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
Tom> Hm, I was just working on inserting something of the sort into
Tom> ExecInitAgg. But I guess we could do it in the planner too. Will
Tom> run with your approach.
Tom> I think it's a bit too stupid as-is, though. We don't need to
Tom> recalculate for Params in aggdirectargs, do we?
In theory we would need to. But in practice we don't, because we don't
allow ordered aggs in AGG_HASHED mode anyway.
We could skip filling in aggParam at all if not in AGG_HASHED mode I
guess.
--
Andrew (irc:RhodiumToad)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2016-08-24 16:29:10 | Re: Strange result with LATERAL query |
| Previous Message | Robert Haas | 2016-08-24 16:16:13 | Re: [Patch] Temporary tables that do not bloat pg_catalog (a.k.a fast temp tables) |