On Tue, 1 Oct 2024 at 06:17, Laurenz Albe <laurenz.a...@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
