| From: | Melih Mutlu <m(dot)melihmutlu(at)gmail(dot)com> |
|---|---|
| To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com>, "shiy(dot)fnst(at)fujitsu(dot)com" <shiy(dot)fnst(at)fujitsu(dot)com> |
| Cc: | Euler Taveira <euler(at)eulerto(dot)com>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, "osumi(dot)takamichi(at)fujitsu(dot)com" <osumi(dot)takamichi(at)fujitsu(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Allow logical replication to copy tables in binary format |
| Date: | 2023-03-17 12:24:55 |
| Message-ID: | CAGPVpCQDsg3D-dd6rjqt3=DaQ+FAZzzF6zf2Y-6uxAocCWB_EA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
Sharing v17.
Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, 17 Mar 2023 Cum, 03:02 tarihinde
şunu yazdı:
> I think to reduce the risk of breakage, let's change the check to
> >=v16. Also, accordingly, update the doc and commit message.
>
Done.
Peter Smith <smithpb2250(at)gmail(dot)com>, 17 Mar 2023 Cum, 04:58 tarihinde şunu
yazdı:
> IMO the sentence "However, logical replication in binary format is
> more restrictive." should just be plain text.
>
Done.
shiy(dot)fnst(at)fujitsu(dot)com <shiy(dot)fnst(at)fujitsu(dot)com>, 17 Mar 2023 Cum, 05:26
tarihinde şunu yazdı:
> It looks that you forgot to pass `offset` into wait_for_log().
Yes, I somehow didn't include those lines into the patch. Thanks for
noticing. Fixed them now.
Thanks,
--
Melih Mutlu
Microsoft
| Attachment | Content-Type | Size |
|---|---|---|
| v17-0001-Allow-logical-replication-to-copy-table-in-binary.patch | application/octet-stream | 15.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jim Jones | 2023-03-17 12:30:49 | Re: [PATCH] Add CANONICAL option to xmlserialize |
| Previous Message | torikoshia | 2023-03-17 12:23:14 | Re: POC PATCH: copy from ... exceptions to: (was Re: VLDB Features) |