On Tue, Sep 7, 2021 at 11:06 AM Amit Kapila <amit.kapil...@gmail.com> wrote:
> On Mon, Sep 6, 2021 at 11:21 PM Alvaro Herrera <alvhe...@alvh.no-ip.org> > wrote: > > > > On 2021-Sep-06, Rahila Syed wrote: > > > > > > > ... ugh. Since CASCADE is already defined to be a > > > > > potentially-data-loss operation, then that may be acceptable > > > > > behavior. For sure the default RESTRICT behavior shouldn't do it, > > > > > though. > > > > > > > > That makes sense to me. > > > > > > However, the default (RESTRICT) behaviour of DROP TABLE allows > > > removing the table from the publication. I have implemented the > > > removal of table from publication on drop column (RESTRICT) on the > > > same lines. > > > > But dropping the table is quite a different action from dropping a > > column, isn't it? If you drop a table, it seems perfectly reasonable > > that it has to be removed from the publication -- essentially, when the > > user drops a table, she is saying "I don't care about this table > > anymore". However, if you drop just one column, that doesn't > > necessarily mean that the user wants to stop publishing the whole table. > > Removing the table from the publication in ALTER TABLE DROP COLUMN seems > > like an overreaction. (Except perhaps in the special case were the > > column being dropped is the only one that was being published.) > > > > So let's discuss what should happen. If you drop a column, and the > > column is filtered out, then it seems to me that the publication should > > continue to have the table, and it should continue to filter out the > > other columns that were being filtered out, regardless of > CASCADE/RESTRICT. > > > > Yeah, for this case we don't need to do anything and I am not sure if > the patch is dropping tables in this case? > > > However, if the column is *included* in the publication, and you drop > > it, ISTM there are two cases: > > > > 1. If it's DROP CASCADE, then the list of columns to replicate should > > continue to have all columns it previously had, so just remove the > > column that is being dropped. > > > > Note that for a somewhat similar case in the index (where the index > has an expression) we drop the index if one of the columns used in the > index expression is dropped, so we might want to just remove the > entire filter here instead of just removing the particular column or > remove the entire table from publication as Rahila is proposing. > > I think removing just a particular column can break the replication > for Updates and Deletes if the removed column is part of replica > identity. > But how this is specific to this patch, I think the behavior should be the same as what is there now, I mean now also we can drop the columns which are part of replica identity right. -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com