From: | Abbas Butt <abbas(dot)butt(at)enterprisedb(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: PG_TRY & PG_CATCH in FDW development |
Date: | 2017-04-26 04:43:40 |
Message-ID: | CALtH27dz1MAC_rchyh_wGc0R5WT0TibQzH+TWWRmmMmiz4cLXg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Thanks for the reply.
On Tue, Apr 25, 2017 at 7:45 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Abbas Butt <abbas(dot)butt(at)enterprisedb(dot)com> writes:
> > What is happening for me is that PG_RE_THROW takes me to PG_TRY in the
> same
> > function and then PG_TRY jumps to PG_CATCH where PG_RE_THROW again jumps
> to
> > PG_TRY in the same function resulting in an infinite loop. The query
> > therefore never returns. It is supposed to throw the error and quit.
>
> Apparently PG_exception_stack isn't getting restored properly, but it's
> sure hard to see why. I'm suspicious that you have something silly like
> mismatched braces in the vicinity of the TRY/CATCH structure.
>
I rechecked, braces are matching.
>
> FWIW, doing things like disconnecting remote sessions might be better
> handled in transaction-cleanup logic, anyway.
I see that postgres_fdw is using a similar login in pgfdw_xact_callback.
Let me try and use the same technique.
> What covers you for that
> if the query aborts while control is not within your PG_TRY block?
>
All the code that requires a connection to the foreign server is with in
the PG_TRY block, it is therefore not required any where else to close
connection before reporting any error.
>
> regards, tom lane
>
--
--
*Abbas*
Architect
Ph: 92.334.5100153
Skype ID: gabbasb
www.enterprisedb.co <http://www.enterprisedb.com/>m
<http://www.enterprisedb.com/>
*Follow us on Twitter*
@EnterpriseDB
Visit EnterpriseDB for tutorials, webinars, whitepapers
<http://www.enterprisedb.com/resources-community> and more
<http://www.enterprisedb.com/resources-community>
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2017-04-26 04:56:48 | Re: Foreign Join pushdowns not working properly for outer joins |
Previous Message | Amit Kapila | 2017-04-26 04:04:02 | Re: PG 10 release notes |