Hi Jan, > Apart from this, the patch looks good to me.
Thanks for the fast and detailed feedback. I will submit an updated patch shortly. > The assert fires because parseRelOptions() counts every option > registered for the kind, while heap_reloptions() still passes a fixed > parse table, so numoptions exceeds num_relopt_elems, and I did not find > an easy way to fix that. I came to the same conclusion. The idea of supporting extendable per-table options independent of TAM sounds nice though. I will investigate this and propose a separate path, unless anyone believes there is no way Postgres should provide such a possibility. -- Best regards, Aleksander Alekseev
