On Wed, 3 Jul 2024 at 22:41, David Rowley <dgrowle...@gmail.com> wrote: > > On Wed, 3 Jul 2024 at 00:20, Matthias van de Meent > > Lastly, I think this would benefit from a test in > > regress/sql/explain.sql, as the test changes that were included > > removed the only occurrance of a Materialize node from the regression > > tests' EXPLAIN outputs. > > I've modified the tests where the previous patch disabled > enable_material to enable it again and mask out the possibly unstable > part. Do you think that's an ok level of testing?
I spent quite a bit more time today looking at these patches. Mostly that amounted to polishing comments more. I've pushed them both now. Thank you both of you for reviewing these. David
