Re: Typo in README.barrier

Tatsuo Ishii <ishii@sraoss.co.jp>

From: Tatsuo Ishii <ishii@sraoss.co.jp>
To: dgrowleyml@gmail.com
Cc: pgsql-hackers@lists.postgresql.org
Date: 2021-05-17T00:33:27Z
Lists: pgsql-hackers
> Thanks for working on that.  I had a look and wondered if it might be
> better to go into slightly less details about the exact atomic
> function to use.  The wording there might lead you to believe you can
> just call the atomic function on the non-atomic variable.
> 
> It might be best just to leave the details about how exactly to use
> atomics by just referencing port/atomics.h.
> 
> Maybe something like the attached?

Thanks. Agreed and your patch looks good to me.

> I'm also a bit on the fence if this should be backpatched or not.  The
> reasons though maybe not is that it seems unlikely maybe people would
> not be working in master if they're developing something new.   On the
> other side of the argument, 0ccebe779, which adjusts another README
> was backpatched.  I'm leaning towards backpatching.

Me too. Let's backpatch.

Best regards,
--
Tatsuo Ishii
SRA OSS, Inc. Japan
English: http://www.sraoss.co.jp/index_en.php
Japanese:http://www.sraoss.co.jp



Commits

  1. Fix typo and outdated information in README.barrier