Hi, On 2019-07-18 11:57:44 +1200, David Rowley wrote: > However, I spent quite a bit of time trying to make that function as > fast as possible in v12, and since #2 seems like a perfectly good > alternative, I'd rather go with that than to add pollution to > ExecFindPartition's signature. Also, #2 seems better since it keeps > CopyFrom() from having to maintain a list. I think we all agreed > somewhere that that code is more complex than we'd all like it to be.
Fair enough. One last thought for #1: I was wondering about is whether a the bool * approach might be useful for nodeModifyTable.c too? I thought that maybe that could be used to avoid some checks for setting up per partition state, but it seems not to be the case ATM. Greetings, Andres Freund