| From: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
|---|---|
| To: | Aleksander Alekseev <aleksander(at)tigerdata(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Andres Freund <andres(at)anarazel(dot)de> |
| Subject: | Re: [PATCH] Remove unused #include's in src/backend/commands/* |
| Date: | 2025-10-09 15:07:03 |
| Message-ID: | aOfPl7alRpClc1bn@nathan |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Oct 09, 2025 at 05:04:16PM +0300, Aleksander Alekseev wrote:
> Thanks for your feedback and apologies for the noise on the mailing
> list. It was not intended.
I wasn't aware of these IWYU pitfalls, sorry. In any case, the reason I
pushed for doing one remove-#includes commit for the whole tree is because
I don't want to start a trend that results in hundreds of commits. Perhaps
this should be treated more like the other cleanup we do prior to creating
a new stable branch. I'm dubious it's worth doing more often than once per
year, if at all.
--
nathan
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Sami Imseih | 2025-10-09 15:07:17 | Re: Add mode column to pg_stat_progress_vacuum |
| Previous Message | Mircea Cadariu | 2025-10-09 15:04:46 | Re: [BUG] temporary file usage report with extended protocol and unnamed portals |