Re: Patch for Race Condition. - Mailing list pgadmin-hackers

From Dinesh Kumar
Subject Re: Patch for Race Condition.
Date
Msg-id CAKWsr7hdX5p4oaSaKh+0XkdDTj7HvEzEvEC80sud3LX=nstK1A@mail.gmail.com
Whole thread Raw
In response to Re: Patch for Race Condition.  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Thanks Dave.

Dinesh

-- 
Dinesh Kumar
Software Engineer

Ph: +918087463317
Skype ID: dinesh.kumar432
www.enterprisedb.com

Follow us on Twitter

@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more


On Fri, Sep 27, 2013 at 8:22 PM, Dave Page <dpage@pgadmin.org> wrote:
Thanks - applied with minor changes to the comments and mutex name.


On Fri, Sep 27, 2013 at 8:51 AM, Dinesh Kumar <dinesh.kumar@enterprisedb.com> wrote:
Hi Dave,

Sorry, it's my bad. The patch is build on another branch.

Please find the attached patch which has been built on the pgAdmin master branch.

Thanks in advance.


Dinesh

-- 
Dinesh Kumar
Software Engineer
Skype ID: dinesh.kumar432
www.enterprisedb.com

Follow us on Twitter

@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more


On Thu, Sep 26, 2013 at 7:54 PM, Dave Page <dpage@pgadmin.org> wrote:



On Tue, Sep 24, 2013 at 12:37 PM, Dinesh Kumar <dinesh.kumar@enterprisedb.com> wrote:
Hi Dave/Team,

While working on one of the assigned task, i have been noticed the race condition on object "currentObject", which is concurrently updating by "onSelRightClick" and "execSelChange" methods. Due to this, sometimes, i am getting "n" Node context menu on "m" node.

Ex:- When i right click on "server node", it is showing the "database node" popup menu options.

I have been attaching the patch along with this request, and will be awaiting for your inputs.

This doesn't apply - can you check the patch is correct please? 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company




--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: Patch for Race Condition.
Next
From: Guillaume Lelarge
Date:
Subject: pgAdmin III commit: Update .pot file