Request a Demo
Home   >   mailing lists

Re: Update EnableTimeoutParams timeout type comment - Mailing list pgsql-hackers

From Xuneng Zhou
Subject Re: Update EnableTimeoutParams timeout type comment
Date
Msg-id CABPTF7WtRLaVKq9WUh=2adBSFGfyAmBdJCFc-PxYDRzndDMUqQ@mail.gmail.com
Whole thread
List pgsql-hackers
Tree view 
On Wed, Jul 1, 2026 at 9:43 AM Xuneng Zhou <xunengzhou@gmail.com> wrote:
>
> Hi Hackers,
>
> While reading the timeout code, I noticed that the comment for
> EnableTimeoutParams timeout type is kinda stale. Currently, we support
> three types of timeout TMPARAM_AFTER/AT/EVERY, but the comment only
> mentions the first two. It may not cause confusion, since the two
> structs are adjacent, but it might still be worth making this clearer.
> Attached a patch to update it.

Opus, it should be "the struct and the enum", not two structs.

--
Regards,
Xuneng Zhou
HighGo Software Co., Ltd.

pgsql-hackers by date:

Previous
From: Tender Wang
Date:
Subject: Re: Fix HAVING-to-WHERE pushdown with mismatched operator families
Next
From: wenhui qiu
Date:
Subject: Re: WAL compression setting after PostgreSQL LZ4 default change