On Tue, 1 Oct 2024 at 06:17, Laurenz Albe <laurenz.albe@cybertec.at> wrote:
> Why did you change "Disabled" from an integer to a boolean?
I just don't think "Disabled Nodes" is all that self-documenting and
I'm also unsure why the full integer value of disabled_nodes is
required over just displaying the boolean value of if the node is
disabled or not. Won't readers look at the remainder of the plan to
determine information about which other nodes are disabled? Do we need
to give them a running total?
> If you see a join where two plans were disabled, that's useful information.
I'm not sure if I follow what you mean here. The patch will show
"Disabled: true" for both the inner and outer side of the join if both
of those are disabled. The difference is that my patch does not show
the join itself is disabled like master does. I thought that's what
you were complaining about. Can you show an example of what you mean?
David
