On Wed, 23 Jul 2025 at 23:08, Ilia Evdokimov <ilya.evdoki...@tantorlabs.com> wrote: > Given that, patch v11 still looks like the most appropriate version to me.
I spent a bit more time cleaning a few things up in this. With all the complexity from the extra fields, I felt that a round of adjusting the "double" types so they use the "Cardinality" typedef where appropriate was in order. costsize.c isn't very good at doing that, but I didn't see that as a reason to make the problem worse. The only other change was in explain.c where I added a local "Memoize *" variable rather than casting the "plan" variable to the subclass type in each usage. I understand not everyone got what they wanted here, but I hope what's been added is ok for the majority of people. Thanks for working on this Ilia and Lukas. David
