|From:||Alik Khilazhev <a(dot)khilazhev(at)postgrespro(dot)ru>|
|To:||Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>|
|Cc:||PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: [HACKERS] [WIP] Zipfian distribution in pgbench|
|Views:||Raw Message | Whole Thread | Download mbox|
Sorry for not responding for long time.
> 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?
> Maybe the cache overflow could be counted, to allow for a possible warning message in the final report?
Also done. But one question: Should it be done by printing to stdout or stderr ?
> When/if the pgbench tap test patch get through, coverage tests should
> be added.
I have added few tests and I am not sure if they OK or not. It was my first experience with perl and tests in PostgreSQL.
|Next Message||Emre Hasegeli||2017-11-19 20:10:38||Re: [HACKERS] [PATCH] Improve geometric types|
|Previous Message||Tom Lane||2017-11-19 19:22:31||Re: Patch to add dependency between client executes and static libraries|