Re: Custom connstr in background_psql()

Lists: pgsql-hackers
From: a(dot)imamov(at)postgrespro(dot)ru
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Custom connstr in background_psql()
Date: 2024-09-17 22:08:26
Message-ID: [email protected]
Views: Whole Thread | Raw Message | Download mbox | Resend email
Lists: pgsql-hackers

Hi, hackers!

I've noticed that there is no way to specify a custom connection string
when
calling the PostgreSQL::Test::Cluster->background_psql() method compared
to the
PostgreSQL::Test:Cluster->psql(). It seems useful to have this feature
while
testing with BackgroundPsql, for example, when the default host value
needs to
be overridden to establish different types of connections.

What do you think?

Attachment Content-Type Size
background_psql.patch text/x-diff 1.2 KB

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: a(dot)imamov(at)postgrespro(dot)ru
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Custom connstr in background_psql()
Date: 2024-09-17 22:57:11
Message-ID: [email protected]
Views: Whole Thread | Raw Message | Download mbox | Resend email
Lists: pgsql-hackers

On Wed, Sep 18, 2024 at 01:08:26AM +0300, a(dot)imamov(at)postgrespro(dot)ru wrote:
> I've noticed that there is no way to specify a custom connection string when
> calling the PostgreSQL::Test::Cluster->background_psql() method compared to the
> PostgreSQL::Test:Cluster->psql(). It seems useful to have this feature while
> testing with BackgroundPsql, for example, when the default host value needs
> to be overridden to establish different types of connections.
>
> What do you think?

I think that it makes sense to extend the routine as you are
suggesting. At least I can see myself using it depending on the test
suite I am dealing with. So count me in.
--
Michael


From: a(dot)imamov(at)postgrespro(dot)ru
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Custom connstr in background_psql()
Date: 2024-09-19 13:00:30
Message-ID: [email protected]
Views: Whole Thread | Raw Message | Download mbox | Resend email
Lists: pgsql-hackers

Michael Paquier писал(а) 2024-09-18 01:57:
> On Wed, Sep 18, 2024 at 01:08:26AM +0300, a(dot)imamov(at)postgrespro(dot)ru
> wrote:
>> I've noticed that there is no way to specify a custom connection
>> string when
>> calling the PostgreSQL::Test::Cluster->background_psql() method
>> compared to the
>> PostgreSQL::Test:Cluster->psql(). It seems useful to have this feature
>> while
>> testing with BackgroundPsql, for example, when the default host value
>> needs
>> to be overridden to establish different types of connections.
>>
>> What do you think?
>
> I think that it makes sense to extend the routine as you are
> suggesting. At least I can see myself using it depending on the test
> suite I am dealing with. So count me in.
> --
> Michael

Should I register the proposal in CF?
Which one to choose if so?

--
regards,
Aidar Imamov


From: Michael Paquier <michael(at)paquier(dot)xyz>
To: a(dot)imamov(at)postgrespro(dot)ru
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Custom connstr in background_psql()
Date: 2024-09-19 22:53:13
Message-ID: [email protected]
Views: Whole Thread | Raw Message | Download mbox | Resend email
Lists: pgsql-hackers

On Thu, Sep 19, 2024 at 04:00:30PM +0300, a(dot)imamov(at)postgrespro(dot)ru wrote:
> Should I register the proposal in CF?
> Which one to choose if so?

I think that's useful, so I'll just go apply it.
--
Michael