On Wed, 2024-10-02 at 21:31 +0300, Alena Rybakina wrote: > Honestly, I like this patch. Before this patch, when disabling any algorithm > in the optimizer, the cost increased significantly and I’m not sure that this > was a reliable solution due to the fact that the cost even without disabling > can be greatly increased because of the high cardinality, for example. > > Right there, the mechanism is simple and more honest in my opinion - we simply > count the number of disabled nodes and discard the paths with the largest > number of them.
I have no issue with this way of handling disabled plan nodes, I only complained about the verbosity of the EXPLAIN output. I don't want to see disabled nodes propagated all the way up the tree, and I would like the output suppressed by default. Yours, Laurenz Albe
