Patch v88-0001 LGTM.
Below are just some minor review comments about the commit message.
======
Commit message
1.
We have discussed having this parameter as a subscription option but
exposing a parameter that is primarily used for testing/debugging to users
didn't seem advisable and there is no other such parameter. The other
option we have discussed is to have a separate GUC for subscriber-side
testing but it appears that for the current testing existing parameter is
sufficient and avoids adding another GUC.
SUGGESTION
We discussed exposing this parameter as a subscription option, but it
did not seem advisable since it is primarily used for
testing/debugging and there is no other such developer option.
We also discussed having separate GUCs for publisher/subscriber-side,
but for current testing/debugging requirements, one GUC is sufficient.
~~
2.
Reviewed-by: Pater Smith, Kuroda Hayato, Amit Kapila
"Pater" --> "Peter"
------
Kind Regards,
Peter Smith.
Fujitsu Australia
