Dear Katsuragi-san,

Thank you for reading the patch! PSA new version.

> Thank you for updating the patch!
> 
> +/* Check whether the postgres server is still alive or not */
> +extern int PQConnCheck(PGconn *conn);
> +extern int PQCanConnCheck(void);
> 
> Aren't these PQconnCheck and PQcanConnCheck? I think the first letter
> following 'PQ' should be lower case.

I cannot find any rules about it, but seems right. Fixed.

Best Regards,
Hayato Kuroda
FUJITSU LIMITED

Attachment: v27-0001-Add-PQconnCheck-and-PQcanConnCheck-to-libpq.patch
Description: v27-0001-Add-PQconnCheck-and-PQcanConnCheck-to-libpq.patch

Attachment: v27-0002-postgres_fdw-add-postgres_fdw_verify_connection_.patch
Description: v27-0002-postgres_fdw-add-postgres_fdw_verify_connection_.patch

Attachment: v27-0003-add-test.patch
Description: v27-0003-add-test.patch

Reply via email to