Re: query cancel issues in contrib/dblink - Mailing list pgsql-hackers

From Merlin Moncure
Subject Re: query cancel issues in contrib/dblink
Date
Msg-id b42b73150906260700h5d09593fx18afb1f9524329a5@mail.gmail.com
Whole thread Raw
In response to query cancel issues in contrib/dblink  (Itagaki Takahiro <itagaki.takahiro@oss.ntt.co.jp>)
Responses Re: query cancel issues in contrib/dblink  (Itagaki Takahiro <itagaki.takahiro@oss.ntt.co.jp>)
List pgsql-hackers
On Thu, Jun 25, 2009 at 10:41 PM, Itagaki
Takahiro<itagaki.takahiro@oss.ntt.co.jp> wrote:
> Hi,
>
> contrib/dblink seems to have no treatments for query cancels.
> It causes the following issues:
>
> (1) Users need to wait for completion of remote query.
>    Requests for query cancel won't be delivered to remote servers.
>
> (2) PGresult objects will be memory leak. The result is not released
>    when query is cancelled; it is released only when dblink function
>    is called max_calls times.
>
> They are long standing issues (not only in 8.4),
> but I hope we will fix them to make dblink more robust.
>
> For (1), asynchronous libpq functions should be used instead of blocking
> ones, and wait for the remote query using a loop with CHECK_FOR_INTERRUPTS().

How would you structure this loop exactly?

merlin


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [PATCH] backend: compare word-at-a-time in bcTruelen
Next
From: Peter Eisentraut
Date:
Subject: Re: Proposal: More portable way to support 64bit platforms