On Friday, March 27, 2026 2:20 PM Masahiko Sawada <[email protected]> wrote: > I've attached the updated patch. I believe I've addressed all comments I got > so far. In addition to that, I've refactored > is_table_publishable_in_publication() and added more regression tests.
Thanks for updating the patch. The latest patch looks mostly good to me. However, I noticed one issue: the function returns table information even for unlogged or temporary tables. I think we should return NULL for those cases instead. BTW, I think we could use is_publishable_class() as a reference to check once whether all unpublishable table types are properly ignored in this function. Best Regards, Hou zj
