From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Refactoring SSL tests |
Date: | 2022-02-08 14:24:13 |
Message-ID: | 2F0D52EA-DC98-4DE6-86FF-CF168D26A1C6@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 7 Feb 2022, at 17:29, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
> On 2/2/22 14:50, Daniel Gustafsson wrote:
>>> On 2 Feb 2022, at 17:09, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>>> On 2/2/22 08:26, Daniel Gustafsson wrote:
>>>> Thoughts? I'm fairly sure there are many crimes against Perl in this patch,
>>>> I'm happy to take pointers on how to improve that.
>>> It feels a bit odd to me from a perl POV. I think it needs to more along
>>> the lines of standard OO patterns. I'll take a stab at that based on
>>> this, might be a few days.
>> That would be great, thanks!
>
> Here's the result of that surgery. It's a little incomplete in that it
> needs some POD adjustment, but I think the code is right - it passes
> testing for me.
Confirmed, it passes all tests for me as well.
> One of the advantages of this, apart from being more idiomatic, is that
> by avoiding the use of package level variables you can have two
> SSL::Server objects, one for OpenSSL and (eventually) one for NSS. This
> was the original motivation for the recent install_path additions to
> PostgreSQL::Test::Cluster, so it complements that work nicely.
Agreed, this version is a clear improvement over my attempt. Thanks!
The attached v2 takes a stab at fixing up the POD sections.
--
Daniel Gustafsson https://vmware.com/
Attachment | Content-Type | Size |
---|---|---|
v2-0001-ssl-test-refactoring.patch | application/octet-stream | 44.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Sharma | 2022-02-08 14:57:32 | Re: Synchronizing slots from primary to standby |
Previous Message | Fujii Masao | 2022-02-08 14:16:38 | Re: [PATCH] Add min() and max() aggregate functions for xid8 |