On 2022/07/07 16:26, Kyotaro Horiguchi wrote:
+ * ControlFileLock is not required as we are the only
+                                                * updator of these variables.

Isn't it better to add "at this time" or something at the end of the
comment because only we're not always updator of them? No?

Excluding initialization, (I believe) checkpointer is really the only
updator of the variables/members.  But I'm fine with the addition.

Ok, so I modified the patch slightly and pushed it. Thanks!

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION


Reply via email to