Re: pgbench more operators & functions

From: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
To: Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>
Cc: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Stephen Frost <sfrost(at)snowman(dot)net>, Jeevan Ladhe <jeevan(dot)ladhe(at)enterprisedb(dot)com>, PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgbench more operators & functions
Date: 2016-12-02 06:28:20
Message-ID: alpine.DEB.2.20.1612020716230.8203@lancre
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Hello Haribabu,

>> Alas, performance testing is quite sensitive to many details:-(

> The current status of the patch and recent mail thread discussion doesn't
> represent the same.

The same what?

The discussion was about a particular test in a particular setting for a
particular load, the fact that reducing the latency has a limited effect
in that case is a fact in life. I have produced other settings where the
effect was very important. The patch has no down side AFAICS.

> Closed in 2016-11 commitfest with "returned with feedback" status.
> Please feel free to update the status once you submit the updated patch.

Given the thread discussions, I do not understand why this "ready for
committer" patch is switched to "return with feedback", as there is
nothing actionnable, and I've done everything required to improve the
syntax and implementation, and to justify why these functions are useful.

I'm spending time to try to make something useful of pgbench, which
require a bunch of patches that work together to improve it for new use
case, including not being limited to the current set of operators.

This decision is both illogical and arbitrary.

--
Fabien.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Haribabu Kommi 2016-12-02 06:29:27 Re: pg_sequence catalog
Previous Message Amit Kapila 2016-12-02 06:22:22 Re: Parallel execution and prepared statements