Horton

Thread

Commits

  1. Fix pgbench's --progress-timestamp option to print Unix-epoch timestamps.

  2. Use clock_gettime(), if available, in instr_time measurements.

  1. pgbench --progress-timestamp no longer works correctly

    Jeff Janes <jeff.janes@gmail.com> — 2017-04-07T16:58:07Z

    --progress-timestamp is supposed to make -P report a Unix Epoch time stamp,
for easy correlation with the entries in other log files (like the postgres
server log file using %n).

But that broke in this commit:

commit 1d63f7d2d180c8708bc12710254eb7b45823440f
Author: Tom Lane <tgl@sss.pgh.pa.us>
Date:   Mon Jan 2 13:41:51 2017 -0500

    Use clock_gettime(), if available, in instr_time measurements.


The commit before that one changed pgbench to make it tolerate the change
in clock, but it overlooked --progress-timestamp.

Cheers,

Jeff

  2. Re: pgbench --progress-timestamp no longer works correctly

    Noah Misch <noah@leadboat.com> — 2017-04-11T04:23:52Z

    On Fri, Apr 07, 2017 at 09:58:07AM -0700, Jeff Janes wrote:
> --progress-timestamp is supposed to make -P report a Unix Epoch time stamp,
> for easy correlation with the entries in other log files (like the postgres
> server log file using %n).
> 
> But that broke in this commit:
> 
> commit 1d63f7d2d180c8708bc12710254eb7b45823440f
> Author: Tom Lane <tgl@sss.pgh.pa.us>
> Date:   Mon Jan 2 13:41:51 2017 -0500
> 
>     Use clock_gettime(), if available, in instr_time measurements.
> 
> 
> The commit before that one changed pgbench to make it tolerate the change
> in clock, but it overlooked --progress-timestamp.

[Action required within three days.  This is a generic notification.]

The above-described topic is currently a PostgreSQL 10 open item.  Tom,
since you committed the patch believed to have created it, you own this open
item.  If some other commit is more relevant or if this does not belong as a
v10 open item, please let us know.  Otherwise, please observe the policy on
open item ownership[1] and send a status update within three calendar days of
this message.  Include a date for your subsequent status update.  Testers may
discover new open items at any time, and I want to plan to get them all fixed
well in advance of shipping v10.  Consequently, I will appreciate your efforts
toward speedy resolution.  Thanks.

[1] https://www.postgresql.org/message-id/20170404140717.GA2675809%40tornado.leadboat.com

  3. Re: pgbench --progress-timestamp no longer works correctly

    Tom Lane <tgl@sss.pgh.pa.us> — 2017-04-11T11:36:26Z

    Noah Misch <noah@leadboat.com> writes:
> On Fri, Apr 07, 2017 at 09:58:07AM -0700, Jeff Janes wrote:
>> --progress-timestamp is supposed to make -P report a Unix Epoch time stamp,
>> for easy correlation with the entries in other log files (like the postgres
>> server log file using %n).
>> 
>> But that broke in this commit:
>> 
>> commit 1d63f7d2d180c8708bc12710254eb7b45823440f
>> Author: Tom Lane <tgl@sss.pgh.pa.us>
>> Date:   Mon Jan 2 13:41:51 2017 -0500

> The above-described topic is currently a PostgreSQL 10 open item.  Tom,
> since you committed the patch believed to have created it, you own this open
> item.  If some other commit is more relevant or if this does not belong as a
> v10 open item, please let us know.  Otherwise, please observe the policy on
> open item ownership[1] and send a status update within three calendar days of
> this message.  Include a date for your subsequent status update.  Testers may
> discover new open items at any time, and I want to plan to get them all fixed
> well in advance of shipping v10.  Consequently, I will appreciate your efforts
> toward speedy resolution.  Thanks.

I'll take a look, and either fix it or post another update before the
end of the week.

			regards, tom lane

  4. Re: pgbench --progress-timestamp no longer works correctly

    Tom Lane <tgl@sss.pgh.pa.us> — 2017-04-11T13:02:51Z

    Jeff Janes <jeff.janes@gmail.com> writes:
> --progress-timestamp is supposed to make -P report a Unix Epoch time stamp,
> for easy correlation with the entries in other log files (like the postgres
> server log file using %n).

> But that broke in this commit:

> commit 1d63f7d2d180c8708bc12710254eb7b45823440f
> Author: Tom Lane <tgl@sss.pgh.pa.us>
> Date:   Mon Jan 2 13:41:51 2017 -0500

>     Use clock_gettime(), if available, in instr_time measurements.

Fixed, thanks.  I suspect I missed this spot because it randomly used
INSTR_TIME_GET_MILLISEC instead of INSTR_TIME_GET_DOUBLE; but for
whatever reason, I missed it.

(So presumably, this option never worked properly on Windows ... but
it will now.)

			regards, tom lane