Re: ecpg PREPARE is not thread safe

From: Michael Meskes <meskes(at)postgresql(dot)org>
To: ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: ecpg PREPARE is not thread safe
Date: 2007-09-21 10:45:18
Message-ID: 20070921104518.GA25527@feivel.credativ.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On Fri, Sep 21, 2007 at 11:05:47AM +0900, ITAGAKI Takahiro wrote:
> PREPARE statements concurrently in several threads. The cause seems to
> be the global variable 'prep_stmts' in ecpg/ecpglib/prepare.c .
> It is accessed without any locks.

And it is global, right. This has to be fixed, you're right.

> I think the proper approach is 3, because server-side prepared statements
> are independent in each connection. For that matter, are there any problems

Right now the prepared statements are not considered connection
specific. I'm not sure whether the standard says anything about this.
But moving this data shoudln't be a major problem.

> Even if we have some kinds of exclusive controls, current ecpg might not
> good at prepared statements when we use multiple connections in a signle
> thread or do multiple PREPARE in multiple threads. If so, 1 and 2 are not
> correct fixes.

Sorry, I don't get this. What exactly are you talking about here?

Michael
--
Michael Meskes
Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes(at)jabber(dot)org
Go SF 49ers! Go Rhein Fire! Use Debian GNU/Linux! Use PostgreSQL!

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Meskes 2007-09-21 11:00:15 Re: compiler warnings in ecpglib/execute.c (uninitialized local variable 'prepname' used)
Previous Message Heikki Linnakangas 2007-09-21 10:24:43 Re: Improving the Performance of Full Table Updates

Browse pgsql-patches by date

  From Date Subject
Next Message ITAGAKI Takahiro 2007-09-21 11:25:02 Re: ecpg PREPARE is not thread safe
Previous Message Gregory Stark 2007-09-21 10:00:31 Re: like/ilike improvements