On Wed, Feb 18, 2026 at 10:09 AM Nisha Moond <[email protected]> wrote:
>
> On Tue, Feb 17, 2026 at 4:44 PM Amit Kapila <[email protected]> wrote:
> >
> > As the worker type is not directly available in all places, the other
> > possibility is to expose a function like GetLogicalWorkerType from
> > worker_internal.h and use it directly in
> > pgstat_report_subscription_error() instead of passing it via
> > parameter. We can get the worker_type via MyLogicalRepWorker->type
> > which should be accessible as it will be invoked by one of the logical
> > replication workers.
> >
> +1
> This seems like a cleaner approach than adding multiple worker
> specific checks and functions.
> Here is a patch that implements this idea.
>

LGTM. I'll push this tomorrow unless there are comments/objections on
this patch.

-- 
With Regards,
Amit Kapila.


Reply via email to