From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
---|---|
To: | Andres Freund <andres(at)2ndquadrant(dot)com> |
Cc: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Mitsumasa KONDO <kondo(dot)mitsumasa(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: gaussian distribution pgbench |
Date: | 2014-07-04 09:59:23 |
Message-ID: | alpine.DEB.2.10.1407041150390.24785@sto |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> Yea. I certainly disagree with the patch in it's current state because
> it copies the same 15 lines several times with a two word difference.
> Independent of whether we want those options, I don't think that's going
> to fly.
I liked a simple static string for the different variants, which means
replication. Factorizing out the (large) common part will mean malloc &
sprintf. Well, why not.
>> OTOH, we've almost reached the consensus that supporting gaussian
>> and exponential options in \setrandom. So I think that you should
>> separate those two features into two patches, and we should apply
>> the \setrandom one first. Then we can discuss whether the other patch
>> should be applied or not.
> Sounds like a good plan.
Sigh. I'll do that as it seems to be a blocker...
The caveat that I have is that without these options there is:
(1) no return about the actual distributions in the final summary, which
depend on the threshold value, and
(2) no included mean to test the feature, so the first patch is less
meaningful if the feature cannot be used simply and require a custom
script.
--
Fabien.
From | Date | Subject | |
---|---|---|---|
Next Message | Abhijit Menon-Sen | 2014-07-04 10:04:14 | Re: pg_xlogdump --stats |
Previous Message | Kyotaro HORIGUCHI | 2014-07-04 09:54:04 | Re: Escaping from blocked send() reprised. |