reshke

> I do have a little knowledge about that in my postgraduate dissertation, but I don't have the confidence to fix bugs quickly in Database planner codes. Thats ok, thats...

> Should add test for join with other distribution table. Maybe, but main reason for having LOCAL relation is that they act exactly like pg_catalog relations (so, used only on...

> > Should add test for join with other distribution table. > > Maybe, but main reason for having LOCAL relation is that they act exactly like pg_catalog relations (so,...

> Could you explain more why need create table on master? Because we need to store and access data on master, during planning phase. At the first glance it is...

If one dislikes SQL approach, we can do it another way: using GUC to control transformDistributedBy behaviour when no distrib clause specified

> > We also can execute heap_create_with_catalog from extension install script, to create all needed relation as we want (with NULL gp_policy > > That is forbidden in GPDB, all...

please remove f342b67c879c7ca9b4f6c6816cd4b974a40060e7 from PR

We are short in time for real production client to start to use this feature, so I propose to incompatibly break all interfaces and rewrite NextVal qdb/protos method to use...

> aqumv_allow_foreign_table PostgreSQL-style GUC would have name like enable_XXX, huh? So, maybe enable_aqumv_foreign_table

> Hi, > > Attached patch fixes the problem: > > [logerrors-pg17-fix.patch.txt](https://github.com/user-attachments/files/16428320/logerrors-pg17-fix.patch.txt) Hi! Thanks for taking care. https://github.com/munakoiso/logerrors/pull/28 merged, should be OK now