Re: Binary support for pgoutput plugin

> On Mon, Nov 11, 2019 at 11:15:45AM -0500, Dave Cramer wrote:
> On Fri, 8 Nov 2019 at 11:20, Dmitry Dolgov <9erthalion6(at)gmail(dot)com> wrote:
>
> > > On Tue, Nov 05, 2019 at 07:16:10AM -0500, Dave Cramer wrote:
> > >
> > > See attached
> >
> > --- a/src/backend/replication/logical/worker.c
> > +++ b/src/backend/replication/logical/worker.c
> > @@ -1779,6 +1779,7 @@ ApplyWorkerMain(Datum main_arg)
> > options.slotname = myslotname;
> > options.proto.logical.proto_version = LOGICALREP_PROTO_VERSION_NUM;
> > options.proto.logical.publication_names =
> > MySubscription->publications;
> > + options.proto.logical.binary = MySubscription->binary;
> >
> > I'm a bit confused, shouldn't be there also
> >
> > --- a/src/backend/catalog/pg_subscription.c
> > +++ b/src/backend/catalog/pg_subscription.c
> > @@ -71,6 +71,7 @@ GetSubscription(Oid subid, bool missing_ok)
> > sub->name = pstrdup(NameStr(subform->subname));
> > sub->owner = subform->subowner;
> > sub->enabled = subform->subenabled;
> > + sub->binary = subform->subbinary;
> >
> > in the GetSubscription?
> >
>
> yes, I have added this. I will supply an updated patch later.
>
> Now a bigger question(s).

Well, without this change it wasn't working for me at all. Other than
that yes, it was a small question :)