> > Oops, you're right. I had failed to notice when reading the code that > the second transaction takes an AccessExclusiveLock on the target > partition. Reverted back to how this was in v3. > > > Also I'd recommend removing the link to a discussion from the test. > Anyway we have link in a commit message. > > -- Report: > https://postgr.es/m/OS3PR01MB5718DA1C4609A25186D1FBF194089%40OS3PR01MB5718.jpnprd01.prod.outlook.com > > Yeah, maybe the link is unnecessary in the test comment, so removed. > Though, I do occasionally see one of those in the test files (try `git > grep https src/test`). >
Thanks! I don't see problems anymore. The patch is RFC now. -- Best regards, Pavel Borisov Postgres Professional: http://postgrespro.com <http://www.postgrespro.com>
