Add support for transition tables #6901
Conversation
Intial go at adding support for transition tables and relaxing the trigger restrictions on AFTER FOR EACH STATEMENT for hypertables Seems to work with both INSERT and COPY (copy uses CIM_SINGLE like in the Postgres source)
There was a problem hiding this comment.
Need to fix this. Can someone who knows what this does comment pls?
There was a problem hiding this comment.
That Assert is weird cause we are checking for the same thing a line below. What is rel here? I'm guessing your rel is not a hypertable and therefore doesn't have fdw_private but might look like a hypertable depending how you look.
There was a problem hiding this comment.
You probably need to modify the relation classification code to treat the transition table not like a normal hypertable.
|
In the pg source there is explicit comment blocking transition tables for partitioned tables did you investigate the reason for this? I think you might have to do attribute mapping if we decide to allow this. This also needs rebase since PG13 support has been removed. |
Intial go at adding support for transition tables and relaxing the trigger restrictions on AFTER FOR EACH STATEMENT for hypertables
Seems to work with both INSERT and COPY (copy uses CIM_SINGLE like in the Postgres source).
This still needs testing on other versions (I did 15), and still needs to have some tests written.
fixes #1084