From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Peter Smith <smithpb2250(at)gmail(dot)com> |
Cc: | "houzj(dot)fnst(at)fujitsu(dot)com" <houzj(dot)fnst(at)fujitsu(dot)com>, shveta malik <shveta(dot)malik(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, "wangw(dot)fnst(at)fujitsu(dot)com" <wangw(dot)fnst(at)fujitsu(dot)com>, "shiy(dot)fnst(at)fujitsu(dot)com" <shiy(dot)fnst(at)fujitsu(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Dilip Kumar <dilipbalaut(at)gmail(dot)com> |
Subject: | Re: Perform streaming logical transactions by background workers and parallel apply |
Date: | 2023-01-13 04:28:18 |
Message-ID: | CAA4eK1L1R=tsaZngquc2fx=JPWAMc3COWXiEk77H8EcPr_8t3Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Jan 13, 2023 at 9:06 AM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> On Fri, Jan 13, 2023 at 7:56 AM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
> >
>
> >
> > 3.
> >
> > <entry role="catalog_table_entry"><para role="column_definition">
> > + <structfield>leader_pid</structfield> <type>integer</type>
> > + </para>
> > + <para>
> > + Process ID of the leader apply worker if this process is a parallel
> > + apply worker; NULL if this process is a leader apply worker or does not
> > + participate in parallel apply, or a synchronization worker
> > + </para></entry>
> >
> > I felt this change is giving too many details and ended up just
> > muddying the water.
> >
>
> I see that we give a similar description for other parameters as well.
> For example leader_pid in pg_stat_activity,
>
BTW, shouldn't we update leader_pid column in pg_stat_activity as well
to display apply leader PID for parallel apply workers? It will
currently display for other parallel operations like a parallel
vacuum, so I don't see a reason to not do the same for parallel apply
workers.
--
With Regards,
Amit Kapila.
From | Date | Subject | |
---|---|---|---|
Next Message | Corey Huinker | 2023-01-13 04:50:26 | Re: Add SHELL_EXIT_CODE to psql |
Previous Message | Justin Pryzby | 2023-01-13 04:17:55 | Re: Cygwin cleanup |