On Wed, Dec 16, 2015 at 3:33 AM, Kyotaro HORIGUCHI
<horiguchi.kyot...@lab.ntt.co.jp> wrote:
> Hello, I think I found a typo in a comment of syncrep.c.
>>   * acknowledge the commit nor raise ERROR or FATAL.  The latter would
>> - * lead the client to believe that that the transaction aborted, which
>>   * is not true: it's already committed locally. The former is no good
> The 'that' looks duplicate.


> And it might be better to put a
> be-verb before the 'aborted'.
>> + * lead the client to believe that the transaction is aborted, which

No, that's correct the way it is.  What you're proposing wouldn't
exactly be wrong, but it's a little less clear and direct.

Committed the part of your patch that removes the extra "that".

Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to