On 2023-09-25 14:38, Michael Paquier wrote:
We would not wait on the lock if force=false, which would do
nowait=true. And !force reads the same to me as force=false.
Anyway, I am OK to remove this part. That seems to confuse you, so
you may not be the only one who would read this comment.
When I first read it, I didn't read that !force as force=false, so
removing it might be better.
Another idea would be to do like in pgstat.c by adding the following
line, then use "nowait" to call each sub-function:
nowait = !force;
pgstat_flush_wal(nowait);
pgstat_flush_io(nowait);
That's very clear and I think it's good.
Ryoga Yoshida