On Mon, Sep 21, 2020 at 6:27 PM Ashutosh Sharma <ashu.coek88@gmail.com> wrote:
>
> Thanks Dilip for the patch. AFAIU, the fix looks good. One small comment:
>
> In the error message we are still referring to the native protocol
> version number. Shouldn't it be replaced with the greatest protocol
> version number we support now (i.e. LOGICALREP_PROTO_MAX_VERSION_NUM)?
>
> - if (data->protocol_version > LOGICALREP_PROTO_VERSION_NUM)
> + if (data->protocol_version > LOGICALREP_PROTO_MAX_VERSION_NUM)
> ereport(ERROR,
> (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
> errmsg("client sent proto_version=%d but we only
> support protocol %d or lower",
> data->protocol_version, LOGICALREP_PROTO_VERSION_NUM)));
>
> Other than this, I don't have any comments.
Thanks for the review. I have fixed it the attached patch.
--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com