Re: 024_add_drop_pub.pl might fail due to deadlock

On Tue, Jul 15, 2025 at 2:24 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
>
> Couple of comments:
> 1) This change is not required:
> #include "utils/snapmgr.h"
> #include "utils/syscache.h"
> #include "utils/usercontext.h"
> +#include "utils/injection_point.h"
>
> 2) This can not only happen in error case but also in normal cases
> where the tablesync worker is slower as shown in the script to
> reproduce, we can update the commit message accordingly:
> In most situations the tablesync worker will drop the corresponding
> origin before it
> finishes executing, but if an error causes the tablesync worker to
> fail just prior to
> dropping the origin, the apply worker will later find the origin and drop it.

Thanks for the test and confirming the fix. Fixed the comments.

regards,
Ajin Cherian
Fujitsu Australia