All I meant was:
doing_sample = trk->ops % cfg->sample_rate == 0;
if (doing_sample)
...
if (doing_sample)
...




On Wed, Nov 27, 2013 at 3:24 PM, Alex Gorrod <notifications@github.com>wrote:

> In bench/wtperf/wtperf.c:
>
> > @@ -241,6 +242,9 @@ static int	 start_threads(
> >  		}
> >
> >  		sprintf(key_buf, "%0*" PRIu64, cfg->key_sz, next_val);
> > +		if (trk->ops % cfg->sample_rate == 0)
>
> Hi @sueloverso <https://github.com/sueloverso>
> On 28 Nov 2013, at 12:58 am, sueloverso <notifications@github.com> wrote:
> In bench/wtperf/wtperf.c: > @@ -241,6 +242,9 @@ static int start_threads( >
> } > > sprintf(key_buf, "%0*" PRIu64, cfg->key_sz, next_val); > + if
> (trk->ops % cfg->sample_rate == 0) The only minor suggestion I have is to
> have a local variable to replace the two places we do the above mod.
> I use the trk->ops field so that we take a fair number of samples from
> each operation type (since the trk structure is specific to an operation
> type). I’d be happy to use a local copy of sample_rate - if that is what
> you mean? - Alex
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/wiredtiger/wiredtiger/pull/774/files#r7968747>
> .
>