Skip site navigation (1) Skip section navigation (2)

Re: connection already closed error

From: Daniele Varrazzo <daniele(dot)varrazzo(at)gmail(dot)com>
To: Mark Theisen <mark(dot)theisen(at)digitecinc(dot)com>
Cc: "psycopg(at)postgresql(dot)org" <psycopg(at)postgresql(dot)org>
Subject: Re: connection already closed error
Date: 2011-10-18 15:11:35
Message-ID: CA+mi_8a3QoF5rLL0QFV2foU=KO+Cc4-jx3L=cZwS8kmUp+OhEw@mail.gmail.com (view raw or flat)
Thread:
Lists: psycopg
On Tue, Oct 18, 2011 at 3:32 PM, Mark Theisen
<mark(dot)theisen(at)digitecinc(dot)com> wrote:

Hi Mark,

> I am using python2.6 with twisted 10.1.0 and psycopg2 2.2.1. We are also using pgpooler with transaction pool mode set. I am using the twisted ConnectionPool and twistedpg.py, which is:

I'm afraid we don't maintain Twisted ConnectionPool. Also, are you are
using a pooling middleware (pgpool) with a python pool (Twisted's)?
This is at best redundant, at worse undefined.

Am I understanding right that "twistedpg" is some module of yours?
Googling for it, it suggest me that I wanted to look for "twisted
pig", which I would prefer to avoid.


> I am getting this error when we restart pgpool:
> [...]
>    conn.close()
> psycopg2.InterfaceError: connection already closed

It doesn't surprise me so much.


> Would a possible solution be to change twistedpg.py to:

>    def close(self):
>        try:
>            _2connection.close(self)
>        except psycopg2.InterfaceError:
>            pass

It seems so. You could also use "if not self.closed:
_2connection.close(self)" (if you have this attribute exposed... why
not just self.close()?). However, I think there is not really a reason
for which conn.close() should raise an error on a closed connection:
it is an idempotent method.

/me looks at the dbapi...

uhm, it doesn't explicitly require that. I think a more robust close()
should just return if the connection is closed. It shouldn't raise
exceptions, as in turn PQfinish doesn't. Oh yes, I see: we used to
execute a ROLLBACK on close() so we needed the connection to be open
to be properly closed... but we don't do this anymore. Currently we do
little more than calling PQfinish, which never fails (well, not
politely at least).

Fog, whaddyathink: "conn.close(); conn.close()" should raise an
exception or should just return?

-- Daniele

In response to

Responses

psycopg by date

Next:From: AMDate: 2011-10-18 15:20:01
Subject: Re: connection already closed error
Previous:From: Mark TheisenDate: 2011-10-18 14:32:13
Subject: connection already closed error

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group