On Thu, Oct 29, 2020 at 8:15 PM Pavel Borisov <pashkin.e...@gmail.com> wrote:
> In case if I'm missing something and Pavel's proposal is significantly >> different from the original patch (if I understand correctly, at the >> moment the latest patch posted here is a rebase and adjusting the old >> patch to work with the latest changes in master, right?), then indeed >> they could be merged, but please in the newer thread [1]. >> > > Sure, my patch has the only difference from the previous Theodor's code > for compatibility with v.13, though it is not small, and I appreciate the > changes in paths processing. The only thing that caused my notice, is that > some useful changes which I've mentioned before, are discarded now. But as > long as they are planned to be put in later it is completely fine. I agree > to discuss the thing in any thread, though I don't quite understand the > reason for a switch. > > Still I don't see a problem. > > -- > Best regards, > Pavel Borisov > > Postgres Professional: http://postgrespro.com <http://www.postgrespro.com> > Regression (stats_ext) is failing because you forgot to drop the table created in a test case (aggregates), It's a bit minor change so the attached patch fixes that issue. https://cirrus-ci.com/task/6704792446697472 -- Ibrar Ahmed
v12-0001-GROUP-BY-optimization-made-compatible-with-chang.patch
Description: Binary data
