On 2021-Sep-15, Amit Kapila wrote:
> On Mon, Sep 6, 2021 at 11:21 PM Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:
> >
> > I pushed the clerical part of this -- namely the addition of
> > PublicationTable node and PublicationRelInfo struct.
>
> One point to note here is that we are developing a generic grammar for
> publications where not only tables but other objects like schema,
> sequences, etc. can be specified, see [1]. So, there is some overlap
> in the grammar modifications being made by this patch and the work
> being done in that other thread.
Oh rats. I was not aware of that thread, or indeed of the fact that
adding multiple object types to publications was being considered.
I do see that 0002 there contains gram.y changes, but AFAICS those
changes don't allow specifying a column list for a table, so there are
some changes needed in that patch for that either way.
I agree that it's better to move forward in unison.
I noticed that 0002 in that other patch uses a void * pointer in
PublicationObjSpec that "could be either RangeVar or String", which
strikes me as a really bad idea. (Already discussed in some other
thread recently, maybe this one or the row filtering one.)
--
Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/
