| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Tommy Pavlicek <tommypav122(at)gmail(dot)com> |
| Cc: | Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com> |
| Subject: | Re: [PATCH] Extend ALTER OPERATOR to support adding commutator, negator, hashes, and merges |
| Date: | 2023-06-23 11:21:46 |
| Message-ID: | 15382.1687519306@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Tommy Pavlicek <tommypav122(at)gmail(dot)com> writes:
> I've added a single patch here: https://commitfest.postgresql.org/43/4389/
> It wasn't obvious whether I should create a second commitfest entry
> because I've included 2 patches so I've just done 1 to begin with. On
> that note, is it preferred here to split patches of this size into
> separate patches, and if so, additionally, separate threads?
No, our commitfest infrastructure is unable to deal with patches that have
interdependencies unless they're presented in a single email. So just use
one thread, and be sure to attach all the patches each time.
(BTW, while you seem to have gotten away with it so far, it's usually
advisable to name the patch files like 0001-foo.patch, 0002-bar.patch,
etc, to make sure the cfbot understands what order to apply them in.)
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Dunstan | 2023-06-23 11:28:52 | Re: Migration database from mysql to postgress |
| Previous Message | Tommy Pavlicek | 2023-06-23 10:34:38 | Re: [PATCH] Extend ALTER OPERATOR to support adding commutator, negator, hashes, and merges |