On Tue, Nov 12, 2024 at 4:02 PM Guillaume Lelarge
<guillaume@lelarge.info> wrote:
> Sure looks easy enough to do (though it still lacks doc and tests changes). See patch attached.
Yep, that's very small. I'm a bit wondering if it's too small, though.
standard_ExplainOneQuery() seems to do some stuff with es->buffers
even before it does planning, so if the idea is that this will be a
noop without ANALYZE, maybe this doesn't implement that. Also, you
should probably update the default value for auto_explain.log_buffers.
In general, I would recommend "git grep 'es->buffers'" and look
carefully at each place where it's mentioned and decide if anything
needs to be changed. And then change the stuff that needs it, and
include in your email an explanation of why the other things don't
need to be changed, unless it's obvious.
--
Robert Haas
EDB: http://www.enterprisedb.com
