Re: [PATCH] Two remaining shmem attachment issues in single-user mode - Mailing list pgsql-hackers

From Chao Li
Subject Re: [PATCH] Two remaining shmem attachment issues in single-user mode
Date
Msg-id 966D70CF-A6FD-48E6-9843-04B87FFF99AE@gmail.com
Whole thread
In response to Re: [PATCH] Two remaining shmem attachment issues in single-user mode  (Jacob Champion <jacob.champion@enterprisedb.com>)
List pgsql-hackers

> On Sep 23, 2026, at 02:52, Jacob Champion <jacob.champion@enterprisedb.com> wrote:
>
> On Sun, Sep 20, 2026 at 1:28 AM Chao Li <li.evan.chao@gmail.com> wrote:
>> Also, for v1-0001, I personally don't feel that the new test is really necessary. This doesn't seem like
particularlyfragile logic that's likely to regress accidentally, 
>
> (But the claim being made is that it _did_ regress accidentally, right?)
>
>> and I've noticed that Tom generally seems reluctant to increase test runtime, even by a very small amount, unless
thetest provides clear value. 
>
> I haven't reviewed the whole patch, but as someone who'd really prefer
> that every bug fix be accompanied by a regression test if possible, I
> don't think we should be discouraging people from writing them. Don't
> waste resources, sure, but don't avoid writing tests. A committer can
> always strip an overly-expensive test out before committing.
>

That’s a good point. Noted.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/







pgsql-hackers by date:

Previous
From: Chao Li
Date:
Subject: Re: [PATCH] Explain what the default output_plugin_libraries do
Next
From: Jeevan Chalke
Date:
Subject: Re: postgres_fdw: push down FETCH FIRST .. WITH TIES when server version allows