From: | Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Tomas Vondra <tomas(dot)vondra(at)postgresql(dot)org>, pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Introduce bloom_filter_size for BRIN bloom opclass |
Date: | 2023-07-03 10:54:08 |
Message-ID: | 30209687-c5f6-e659-67b7-a4972701c601@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 7/3/23 04:18, Andrew Dunstan wrote:
>
> On 2023-07-02 Su 20:30, Tom Lane wrote:
>> Michael Paquier <michael(at)paquier(dot)xyz> writes:
>>> koel has failed its indent step after these commits. I am unsure
>>> whether you need to fix this as it does not seem that we have a
>>> consensus to automate that, just FYI.
>> Well, *somebody* has to fix it, otherwise why did we set up the animal?
>> But the impression I had was that a majority of committers want to
>> start keeping the tree pgindent-clean.
>>
>>
>
>
> I think we need to spell out expectations more clearly. Here's what I
> said to Tomas in a private message:
>
> I think the expectation is that, while there might be cases where
> you want to make some commits and then indent afterwards, so the
> changes are clear, for most cases you should try to commit
> pre-indented patches.
>
> But that's just my opinion. I think we need to spell it out more formally.
>
I've pushed a fix for the indentation issues caused by my commits (this
one and the commit touching trigger.c). But I agree with Andrew it'd be
good to spell out the expectations - perhaps it's spelled out somewhere
deep in the pgindent, not sure.
regards
--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2023-07-03 14:08:44 | pgsql: Improve pg_basebackup long file name test Windows robustness |
Previous Message | Tomas Vondra | 2023-07-03 10:48:32 | pgsql: Fix code indentation violations |