From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Alik Khilazhev <a(dot)khilazhev(at)postgrespro(dot)ru> |
Cc: | PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
Subject: | Re: [WIP] Zipfian distribution in pgbench |
Date: | 2017-10-02 00:43:41 |
Message-ID: | 3E9D138E-2388-41B4-9858-E08F98F0AAC3@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 06 Sep 2017, at 08:42, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> wrote:
>
> Hello Alik,
>
> Applies, compiles, works for me.
>
> Some minor comments and suggestions.
>
> Two typos:
> - "usinng" -> "using"
> - "a rejection method used" -> "a rejection method is used"
>
> I'm not sure of "least_recently_used_i", this naming style is not used in pgbench. "least_recently_used" would be ok.
>
> "..nb_cells.. != ZIPF_CACHE_SIZE", ISTM that "<" is more logical,
> even if the result is the same?
>
> I would put the parameter value check in getZipfianRand, so that if someone reuse the function elsewhere the check is also performed. That would also simplify a bit the already very large expression evaluation function.
>
> When/if the pgbench tap test patch get through, coverage tests should
> be added.
>
> Maybe the cache overflow could be counted, to allow for a possible warning message in the final report?
Since this patch has been Waiting for author and no update on this patch has
been posted during the commitfest, it is Returned with feedback. When you have
a new version of the patch, please re-submit to a future commitfest.
cheers ./daniel
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Langote | 2017-10-02 01:13:17 | Re: path toward faster partition pruning |
Previous Message | Daniel Gustafsson | 2017-10-02 00:36:49 | Re: Small patch for pg_basebackup argument parsing |