On Wed, Aug 24, 2022 at 10:48:01AM +0900, Kyotaro Horiguchi wrote:
> By the way, I think we use INSTR_TIME_* macros to do masure internal
> durations (mainly for the monotonic clock characteristics, and to
> reduce performance degradation on Windows?). I'm not sure that's
> crutial here but I don't think there's any reason to use
> GetCurrentTimestamp() instead.
This implies two calls of gettimeofday(), but that does not worry me
much in this code path. There is some consistency with
CheckpointGuts() where we take timestamps for the sync requests.
--
Michael