Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue with Node rename - Mailing list pgadmin-hackers

From Murtuza Zabuawala
Subject Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue with Node rename
Date
Msg-id CAKKotZRNUt9PAJiqqF9Po4jE9LMXu+VrJi3=SKjFfV+ypxzXRQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue with Node rename  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Responses Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue with Node rename  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
List pgadmin-hackers
Hi Ashesh,

PFA updated patch for the issue.

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Wed, Apr 26, 2017 at 10:29 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Apr 24, 2017 at 4:43 PM, Dave Page <dpage@pgadmin.org> wrote:
Ashesh, can you review/commit this please? Thanks.

On Mon, Apr 24, 2017 at 6:17 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA minor patch to fix the issue where node rename is not working properly after 7dd9efd8 commit .
RM#2355
We should remove the existing node, and then insert at right place instead of refreshing the parent.
Because - that will select the parent node, and not that node, and also - it adds overhead of refreshing the whole parent node.

Please send the patch as per our discussion.

-- Thanks, Ashesh

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers




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

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


Attachment

pgadmin-hackers by date:

Previous
From: Sarah McAlear
Date:
Subject: [pgadmin-hackers][Patch] Rename feature test server
Next
From: Khushboo Vashi
Date:
Subject: Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM #2315 : Sorting bysize is broken