On 05.04.2023 17:35, Tom Lane wrote:
> "Anton A. Melnikov" <aamelnikov@inbox.ru> writes:
>> On 03.04.2023 21:49, Tom Lane wrote:
>>> I did not think this case was worth memorializing in a test before,
>>> and I still do not. I'm inclined to reject this patch.
>
>> Could you help me to figure out, please.
>
> The problem was an Assert that was speculative when it went in,
> and which we eventually found was wrong in the context of logical
> replication. We removed the Assert. I don't think we need a test
> case to keep us from putting back the Assert. That line of thinking
> leads to test suites that run for fourteen hours and are near useless
> because developers can't run them easily.
>
> regards, tom lane
Ok, i understand! Thanks a lot for the clarification. A rather important point,
i'll take it into account for the future.
Let's do that. Revoked the patch.
With the best wishes!
--
Anton A. Melnikov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
