Re: [HACKERS] pgbench more operators & functions - Mailing list pgsql-hackers

From David G. Johnston
Subject Re: [HACKERS] pgbench more operators & functions
Date
Msg-id CAKFQuwZEtyDquOVd03kOM-+0kf6fnb+=K-fb8Hce8aGMXPEDDg@mail.gmail.com
Whole thread Raw
In response to Re: pgbench more operators & functions  (Fabien COELHO <coelho@cri.ensmp.fr>)
Responses Re: [HACKERS] pgbench more operators & functions  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Wed, Oct 5, 2016 at 2:03 AM, Fabien COELHO <coelho@cri.ensmp.fr> wrote:

I've got no objection to a more-nearly-TPC-B script as an option.

Good, because adding a "per-spec" tpc-b as an additional builtin option is one of my intentions, once pgbench is capable of it.

​Trying to scan the thread as an interested third-party - without hacker skills...

Maybe consider writing a full patch series that culminates with this additional builtin option being added as the final patch - breaking out this (and probably other) "requirements" patches ​separately to aid in review.  The presentation of just "add new stuff that seems useful" opens too much room for isolated discussion without knowing what the big picture requires.  At worse you'll at least have a working version of a forked pgbench that can do what you want.

As it stands right now you haven't provided enough context for this patch and only the social difficulty of actually marking a patch rejected has prevented its demise in its current form - because while it has interesting ideas its added maintenance burden for -core without any in-core usage.  But it you make it the first patch in a 3-patch series that implements the per-spec tpc-b the discussion moves away from these support functions and into the broader framework in which they are made useful.

My $0.02

David J.

pgsql-hackers by date:

Previous
From: Corey Huinker
Date:
Subject: Re: \if, \elseif, \else, \endif (was Re: [HACKERS] PSQL commands:\quit_if, \quit_unless)
Next
From: Stephen Frost
Date:
Subject: Re: [HACKERS] [PATCH] Rename pg_switch_xlog to pg_switch_wal