From: | Huong Dangminh <huo-dangminh(at)ys(dot)jp(dot)nec(dot)com> |
---|---|
To: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Cc: | Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>, Hiroshi Yanagisawa <hir-yanagisawa(at)ut(dot)jp(dot)nec(dot)com> |
Subject: | RE: User defined data types in Logical Replication |
Date: | 2017-12-20 05:28:39 |
Message-ID: | 75DB81BEEA95B445AE6D576A0A5C9E936A6DB067@BPXM05GP.gisp.nec.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Sawada-san,
> >> eventually we'll need to map types to local oid (and possibly more)
> >> where the local info is cached so that we can interpret binary
> >> representation of replicated data (which we'll add at some point
> >> since it's big performance boost).
>
> Sounds good.
>
> >>
> >> So I am afraid that if we do the rename of typmap to remotetype in
> >> this patch it will a) make backports of fixes in the related code
> >> harder, b) force us to rename it back again in the future.
> >
> > Thanks for your comment.
> >
> >> I'd keep your general approach but keep using typmap naming.
> >
> > I update the patch as Petr Jelineks mention, keep using typmap naming.
> >
>
> Thank you for updating the patch. Here is a review comment.
Thanks for reviewing.
> - if (errarg->attnum < 0)
> + rel = errarg->rel;
> + remote_attnum = rel->attrmap[errarg->local_attnum];
> +
> + if (remote_attnum < 0)
> return;
>
> I think errarg->local_attnum can be -1 here and access an invalid address
> if slot_store_error_callback() is called before setting
> errarg.local_attnum. I cannot see such call path in the current code so
> far but would need to be fixed.
I updated the patch to fix it.
---
Thanks and best regards,
Dang Minh Huong
NEC Solution Innovators, Ltd.
http://www.nec-solutioninnovators.co.jp/en/
Attachment | Content-Type | Size |
---|---|---|
fix_slot_store_error_callback_v9.patch | application/octet-stream | 7.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-12-20 05:57:12 | Re: [HACKERS] static assertions in C++ |
Previous Message | Michael Paquier | 2017-12-20 05:27:03 | Re: TRAP: FailedAssertion("!(TransactionIdPrecedesOrEquals |