On Wed, Nov 29, 2023 at 7:48 PM Bharath Rupireddy
wrote:
>
> On Wed, Nov 29, 2023 at 1:49 PM Masahiko Sawada wrote:
> >
> > On Wed, Nov 29, 2023 at 4:30 PM Bharath Rupireddy
> > wrote:
> > >
> > > On Tue, Nov 28, 2023 at 12:23 PM Tom Lane wrote:
> > > >
> > > > Bharath Rupireddy writes:
> > >
On Wed, Nov 29, 2023 at 1:49 PM Masahiko Sawada wrote:
>
> On Wed, Nov 29, 2023 at 4:30 PM Bharath Rupireddy
> wrote:
> >
> > On Tue, Nov 28, 2023 at 12:23 PM Tom Lane wrote:
> > >
> > > Bharath Rupireddy writes:
> > > > A nitpick on the patch - how about honoring the passed-in parameter
> > >
On Wed, Nov 29, 2023 at 4:30 PM Bharath Rupireddy
wrote:
>
> On Tue, Nov 28, 2023 at 12:23 PM Tom Lane wrote:
> >
> > Bharath Rupireddy writes:
> > > A nitpick on the patch - how about honoring the passed-in parameter
> > > with something like $self->{query_timer_restart} = 1 if !defined
> > > $
On Tue, Nov 28, 2023 at 12:23 PM Tom Lane wrote:
>
> Bharath Rupireddy writes:
> > A nitpick on the patch - how about honoring the passed-in parameter
> > with something like $self->{query_timer_restart} = 1 if !defined
> > $self->{query_timer_restart}; instead of just setting it to 1 (a value
>
Bharath Rupireddy writes:
> A nitpick on the patch - how about honoring the passed-in parameter
> with something like $self->{query_timer_restart} = 1 if !defined
> $self->{query_timer_restart}; instead of just setting it to 1 (a value
> other than undef) $self->{query_timer_restart} = 1;?
I wond
On Tue, Nov 28, 2023 at 6:48 AM Masahiko Sawada wrote:
>
> Hi,
>
> While adding some TAP tests, I realized that set_query_timer_restart()
> in BackgroundPsql may not work. Specifically, it seems not to work
> unless we pass an argument to the function. Here is the test script I
> used:
>
> If call
