| From: | Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> |
|---|---|
| To: | "movead(dot)li(at)highgo(dot)ca" <movead(dot)li(at)highgo(dot)ca> |
| Cc: | Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, "Andrey V(dot) Lepikhov" <a(dot)lepikhov(at)postgrespro(dot)ru>, Ahsan Hadi <ahsan(dot)hadi(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Asynchronous Append on postgres_fdw nodes. |
| Date: | 2021-01-02 08:15:59 |
| Message-ID: | CAPmGK16A1m=gUXSwvDHnk0w4LY7G6LFYQfK_3=u4P3dEnSAd_g@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sun, Dec 20, 2020 at 5:15 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> On Thu, Nov 26, 2020 at 10:28 AM movead(dot)li(at)highgo(dot)ca
> <movead(dot)li(at)highgo(dot)ca> wrote:
> > Issue one:
> > Get a Assert error at 'Assert(bms_is_member(i, node->as_needrequest));' in
> > ExecAppendAsyncRequest() function when I use more than two foreign table
> > on different foreign server.
> >
> > I research the code and do such change then the Assert problom disappear.
>
> Could you show a test case causing the assertion failure?
I happened to reproduce the same failure in my environment.
I think your change would be correct, but I changed the patch so that
it doesn’t need as_lastasyncplan anymore [1]. The new version of the
patch works well for my case. So, could you test your case with it?
Best regards,
Etsuro Fujita
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Fabien COELHO | 2021-01-02 09:50:15 | Re: Proposed patch for key management |
| Previous Message | Amit Kapila | 2021-01-02 07:36:51 | Re: faster ETL / bulk data load for heap tables |