public inbox for [email protected]  
help / color / mirror / Atom feed
From: Federico Di Gregorio <[email protected]>
To: Daniele Varrazzo <[email protected]>
To: [email protected]
Cc: [email protected]
Subject: Re: Copy command hangs in poll() function
Date: Thu, 4 Oct 2018 12:33:41 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <CA+mi_8Z0EYtyayOYzerwjZww2zFGE=f0oQdvW7P92ZOUvdRhQA@mail.gmail.com>
References: <CANxoLDekyavcxjh1yWk1hhB1ER+YvxUkg1ZP2DOYev+Mr1FULw@mail.gmail.com>
	<CA+mi_8Z0EYtyayOYzerwjZww2zFGE=f0oQdvW7P92ZOUvdRhQA@mail.gmail.com>

On 10/04/2018 12:26 PM, Daniele Varrazzo wrote:
> On Thu, Oct 4, 2018 at 6:35 AM Akshay Joshi 
> <[email protected] <mailto:[email protected]>> 
> wrote:
> 
>     Hello
> 
>     As per documentation http://initd.org/psycopg/docs/advanced.html
>     <http://initd.org/psycopg/docs/advanced.html> Copy commands are not
>     supported in asynchronous mode, but this will be probably
>     implemented in a future release. In pgAdmin4 (open source) we have
>     used async connection to run the SQL query on PostgreSQL database
>     and uses poll() function to poll the results. When user runs any
>     *COPY *command application hangs at poll() function. 
> 
> 
>     If copy command not supported then at least it should not hang the
>     application. poll() function should return the proper error message.
> 
>     Can you please provide that fix until support for copy command won't
>     come.
> 
> 
> Uhm, I see:
> 
>      In [2]: cnn = psycopg2.connect("")
>      In [3]: cur = cnn.cursor()
>      In [4]: cur.execute("copy (select 1 as x) to stdout")
>      
> ---------------------------------------------------------------------------
>      ProgrammingError                          Traceback (most recent 
> call last)
>      <ipython-input-4-62e3af3d121e> in <module>()
>      ----> 1 cur.execute("copy (select 1 as x) to stdout")
>      ProgrammingError: can't execute COPY TO: use the copy_to() method 
> instead
> 
> but:
> 
>      In [6]: cnn = psycopg2.connect("", async=True)
>      In [7]: cnn.poll()
>      Out[7]: 2
>      In [8]: cnn.poll()
>      Out[8]: 1
>      In [9]: cnn.poll()
>      Out[9]: 0
> 
>      In [12]: cur.execute("copy (select 1 as x) to stdout")
>      In [14]: cnn.poll()
>      ^C -- hung
> 
> So yes, I'll take a look if there is a way to detect we get in this 
> state. But I'm not sure it's possible, because we might not have 
> received a result yet that tells we are in a state not allowed (in a COPY).
> 
> Observation: maybe EnterpriseDB has resources enough to help us 
> debugging and fixing this issue, or maybe even implement async copy?

Also, let me add that if the only way to detect the situation is by 
analyzing the query, we'll never do that.

Also, IMHO, it is perfectly fine that a sequence of operations that is 
documented as "not working" hangs - it is not something you can do by 
error: you're ignoring the documentation.

federico

-- 
Federico Di Gregorio                         [email protected]
DNDG srl                                                  http://dndg.it
    I filosofi son come i sociologi: il mondo non lo capiscono. -- A.R.M.




view thread (4+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Copy command hangs in poll() function
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox