RE: Optionally automatically disable logical replication subscriptions on error

Hi Osumi-san,

I have a comment on v21 patch.

I wonder if we really need subscription s2 in 028_disable_on_error.pl. I think
for subscription s2, we only tested some normal cases(which could be tested with s1), 
and didn't test any error case, which means it wouldn't be automatically disabled. 
Is there any reason for creating subscription s2?

Regards,
Tang