Re: pgAgent crashes on failed connection - Mailing list pgadmin-hackers

From Merlin Moncure
Subject Re: pgAgent crashes on failed connection
Date
Msg-id CAHyXU0w7MO=UvGtSjz5z199nHeXuNQxM4HZVJXKGTfwbjtQcMA@mail.gmail.com
Whole thread Raw
In response to pgAgent crashes on failed connection  (Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>)
Responses Re: pgAgent crashes on failed connection
List pgadmin-hackers
On Thu, Aug 4, 2011 at 2:19 PM, Heikki Linnakangas
<heikki.linnakangas@enterprisedb.com> wrote:
> I created 100 identical pgagent jobs, with one step that simply does "SELECT
> pg_sleep(10)". I then forced them all to run immediately, with "UPDATE
> pgagent.pga_job SET jobnextrun=now();". pgagent crashed.
>
> What happened is that the when all those jobs are launched at the same time,
> the server ran into the max_connections limit, and pgagent didn't handle
> that too well. JobThread::JobThread constructor does not check for NULL
> result from DBConn::Get(), and passes a NULL connection to Job::Job, which
> tries to reference it, leading to a segfault.
>
> I propose the attached patch.

hm, in the event that happens, is that logged in the client somehow?
wouldn't you want to throw an exception or something like that?

merlin

pgadmin-hackers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgAgent crashes on failed connection
Next
From: Guillaume Lelarge
Date:
Subject: pgAgent commit: Update HTTP checkout link for INSTALL file