On Mon, Jan 29, 2024 at 10:43 PM Alvaro Herrera <alvhe...@alvh.no-ip.org> wrote: > > Okay, so I gave this another look and concluded that I definitely didn't > like the whole business of having one level open the explain group and > return outwards whether it had been done so that the other level would > close it. So I made the code do what I said I thought it should do > (adding a new function peek_buffer_usage to report whether BUFFERS would > print anything), and I have to say that it looks much better to me with > that.
Hmm. ExplainOnePlan certainly looks better with this. > > I also added a trivial test for EXPLAIN EXECUTE, which was uncovered, > and some other minor stylistic changes. > Thanks. Looks fine to me. > And with that I pushed it. Thanks a lot. -- Best Wishes, Ashutosh Bapat