Hi! Thanks for review!

> So, isn't it better to keep
> all data tables unlogged and define the foreign key from each accounts
> partition to pgbench_branches instead of from the partitioned parent?
> 

Yes, it is... There is no point in keeping the patch simpler at the
cost of the option's meaning, so I did V2 that does:
- keep the PERMANENT exception for pgbench_accounts only: it cannot
  be unlogged when partitioned
- accounts -> branches FKs are now created on the partitions
  (pgbench_accounts_%) if the unlogged option is on and there are
  partitions

--
Best regards,
Roman Khapov

Attachment: v2-0001-pgbench-honor-unlogged-tables-for-non-partitioned.patch
Description: Binary data

Reply via email to