Re: [HACKERS] additional contrib test suites

On 9/14/17 11:01, David Steele wrote:
> On 9/8/17 1:32 PM, Peter Eisentraut wrote:
>>
>> Yes, some of the error messages had changed.  Fixed patches attached.
> 
> Patches apply and all tests pass.  A few comments:
> 
> * [PATCH v2 1/7] adminpack: Add test suite
> 
> There are no regular tests for pg_logdir_ls().

Added a comment about that.

> * [PATCH v2 4/7] chkpass: Add test suite
> 
> Well, this is kind of scary:
> 
> +CREATE EXTENSION chkpass;
> +WARNING:  type input function chkpass_in should not be volatile
> 
> I guess the only side effect is that this column cannot be indexed?  The
> docs say that, so OK, but is there anything else a user should be
> worried about?

Well, we're just testing, not judging. ;-)

> The rest looks good.  I'll mark this "Ready for Committer" since I'm the
> only reviewer.  I don't think anything you might add based on my
> comments above requires a re-review.
> 
> As for testing on more platforms, send it to the build farm?

OK, committed, let's see.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers