Re: PostgreSQL Audit Extension - Mailing list pgsql-hackers

From Noah Misch
Subject Re: PostgreSQL Audit Extension
Date
Msg-id 20160202033118.GA4096289@tornado.leadboat.com
Whole thread Raw
In response to Re: PostgreSQL Audit Extension  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-hackers
On Mon, Feb 01, 2016 at 09:59:01PM +0100, Alvaro Herrera wrote:
> Anyway I think the tests here are massive

I would not want to see fewer tests.  When I reviewed a previous incarnation
of pgaudit, the tests saved me hours of writing my own.

> and the code is not;

Nah; the patch size category is the same regardless of how you arrange the
tests.  The tests constitute less than half of the overall change.

> perhaps people get the mistaken impression
> that this is a huge amount of code which scares them.  Perhaps you could
> split it up in (1) code and (2) tests, which wouldn't achieve any
> technical benefit but would offer some psychological comfort to
> potential reviewers.  You know it's all psychology in these parts.

That's harmless.  If it makes someone feel better, great.



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [PATCH] Phrase search ported to 9.6
Next
From: Noah Misch
Date:
Subject: Re: Raising the checkpoint_timeout limit