Re: too much pgbench init output
Tomas Vondra <tv@fuzzy.cz>
From: Tomas Vondra <tv@fuzzy.cz>
To: pgsql-hackers@postgresql.org
Date: 2012-12-19T21:00:13Z
Lists: pgsql-hackers
Attachments
- pgbench-logging-v5.patch (text/plain) patch v5
On 19.12.2012 06:30, Jeevan Chalke wrote: > > > > On Mon, Dec 17, 2012 at 5:37 AM, Tomas Vondra <tv@fuzzy.cz > <mailto:tv@fuzzy.cz>> wrote: > > Hi, > > attached is a new version of the patch that > > (a) converts the 'log_step_seconds' variable to a constant (and does > not allow changing it using a command-line option etc.) > > (b) keeps the current logging as a default > > (c) adds a "-q" switch that enables the new logging with a 5-second > interval > > I'm still not convinced there should be yet another know for tuning the > log interval - opinions? > > > It seems that you have generated a patch over your earlier version and > due to that it is not cleanly applying on fresh sources. > Please generate patch on fresh sources. Seems you're right - I've attached the proper patch against current master. > However, I absolutely no issues with the design. Also code review is > already done and looks good to me. > I think to move forward on this we need someone from core-team. So I am > planning to change its status to "ready-for-committor". > > Before that please provide updated patch for final code review. thanks Tomas