Re: split tablecmds.c

From: Mario González Troncoso <gonzalemario(at)gmail(dot)com>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: Amit Langote <amitlangote09(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Álvaro Herrera <alvherre(at)kurilemu(dot)de>, pgsql-hackers(at)postgresql(dot)org, euler(at)eulerto(dot)com, andres(at)anarazel(dot)de
Subject: Re: split tablecmds.c
Date: 2026-08-21 21:19:03
Message-ID: CAFsReFXH0h8d+QCcrHatEH-rC5Xa-_dmB+YaO+5MvJo9-7hS=w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, 21 Aug 2026 at 17:15, Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
>
> On Thu, Jan 29, 2026 at 04:17:43PM -0600, Nathan Bossart wrote:
> > Here is a new version of the patch that moves the partitioning code to
> > tablecmds_partition.c and creates a tablecmds_internal.h for the things in
> > tablecmds.c that the partitioning code needs. The patch is still
> > relatively rough.
>
> Here is an attempt at fixing headerscheck.
>

The code needs rebase. I tried to see if I could do it myself but it
seems tablecmds.c has received some modifications that at this point,
I'm not sure what to remove. Some hunk worked but I guess it'll be
better if you do the rebase please. At least I tried:

patching file src/backend/commands/Makefile
patching file src/backend/commands/meson.build
Hunk #1 succeeded at 47 (offset 2 lines).
patching file src/backend/commands/tablecmds.c
Hunk #1 succeeded at 65 with fuzz 1 (offset 3 lines).
Hunk #2 FAILED at 136.
Hunk #3 succeeded at 337 (offset 10 lines).
Hunk #4 succeeded at 351 with fuzz 2 (offset 10 lines).
Hunk #5 succeeded at 459 (offset 45 lines).
Hunk #6 succeeded at 482 (offset 46 lines).
Hunk #7 succeeded at 502 (offset 46 lines).
Hunk #8 succeeded at 531 (offset 46 lines).
Hunk #9 succeeded at 539 (offset 46 lines).
Hunk #10 succeeded at 584 (offset 46 lines).
Hunk #11 succeeded at 594 (offset 46 lines).
Hunk #12 FAILED at 624.
Hunk #13 succeeded at 4462 (offset 102 lines).
Hunk #14 succeeded at 6604 (offset 104 lines).
Hunk #15 succeeded at 6785 (offset 104 lines).
Hunk #16 succeeded at 6916 (offset 120 lines).
Hunk #17 succeeded at 8525 (offset 125 lines).
Hunk #18 succeeded at 10764 (offset 129 lines).
Hunk #19 succeeded at 10942 (offset 129 lines).
Hunk #20 succeeded at 11080 (offset 129 lines).
Hunk #21 succeeded at 11246 (offset 129 lines).
Hunk #22 succeeded at 11256 (offset 129 lines).
Hunk #23 succeeded at 11319 (offset 129 lines).
Hunk #24 succeeded at 12515 (offset 515 lines).
Hunk #25 succeeded at 17086 (offset 712 lines).
Hunk #26 succeeded at 17567 (offset 707 lines).
Hunk #27 succeeded at 17585 (offset 707 lines).
Hunk #28 FAILED at 18650.
Hunk #29 FAILED at 21008.

> --
> nathan

--
Mario Gonzalez

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Haibo Yan 2026-08-21 21:46:01 Re: Checkpointer write combining
Previous Message Bharath Rupireddy 2026-08-21 20:40:00 Re: Further cleanup related to statistics import support in postgres_fdw