Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard
Date
Msg-id CANxoLDfPafNA6nnAVm0wzC487qBBK++f7E_E_x3AFcsZ0v1EMA@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
List pgadmin-hackers
Thanks patch applied.

On Mon, Aug 28, 2017 at 12:27 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

Please find a patch to fix the issue for PG9.6 in Dashboard, this is regression of RM#2597.

Error:
2017-08-28 11:49:28,238: ERROR pgadmin: Failed to execute query (execute_dict) for the server #34- DB:postgres (Query-id: 320468):
Error Message:ERROR:  column "backend_type" does not exist
LINE 14:     backend_type
in dashboard

Earlier I tested changes only with PG9.4 and PG10, So I missed this issue, as "backend_type" column only exists in PG10+.
Thanks to Surinder for reporting the issue.

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

https://community.postgresrocks.net/

On Fri, Aug 25, 2017 at 9:27 PM, Dave Page <dpage@pgadmin.org> wrote:
Thanks, patch applied.

On Wed, Aug 9, 2017 at 12:19 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

Please find the updated patch with new UI style in dashboard subnode control based on Chethana's design suggestion.

@Shirley,
Yes, DBA needs to click on the drop down to expand.
- If we make whole row clickable then how are we going to identify if user is trying to expand the row or trying to cancel the active session? I am asking this because table row <tr> tag comes before table column tag <td> in HTML DOM, so if we have click event listers on both of the tags then how are we going to differentiate the user operation?
- In my opinion, another reason we can not do that on a dashboard grid because it will break the subnode control consistency, If user sees that grid expands on clicking the row they will expect the same behaviour in other grids as well.

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

https://community.postgresrocks.net/

On Tue, Aug 8, 2017 at 7:47 PM, Shirley Wang <swang@pivotal.io> wrote:
Nice! 

Would a DBA need to click on the drop down to expand? I think that drop down icon might be too small. Perhaps the whole row could be a clickable area to show further details. This is a good thing to test with some people


On Aug 8, 2017, at 12:52, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Aug 7, 2017 at 9:06 PM, Dave Page <dpage@pgadmin.org> wrote:


On Mon, Aug 7, 2017 at 3:49 PM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hi Dave,

Could you comment on the new design update for subnode control ?

Attached designs -

1. subnode_current.png -  The existing view
2. subnode_new.png - The updated view
3. current_new.png - Existing and new design placed together for comparison purpose

Below changes done -

1. Reduced number of gray color variations
2. Made more simplified by removing unwanted borders
3. Merged "Arrow Down" background and the body background for more clarity

Please feel free to provide your input on the same.

Much nicer in my opinion! Good work.

Anyone else have any comments?
It looks much nicer.
I would like to see the effect on another dialogs like 'Table properties', where we have multiple level of subgrid controls with tabs.

-- Thanks, Ashesh 
 

Regards,
Chethana kumar



On Fri, Jul 28, 2017 at 4:11 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I took a quick look at this and have a couple of thoughts:

- Instead of the "edit" icon to open the subnode, we should use something more appropriate - a "properties" icon perhaps.

- There seems to be a lot of different shades of grey on there (maybe a subnode design in general that just shows up with the disabled controls), and the subnode control looks a bit messy as a result.

Can you work with Chethana to improve the look and feel please?

Input from others welcome of course - screenshot attached.

Thanks.

On Fri, Jul 28, 2017 at 11:33 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
++ Attaching the patch

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


On Fri, Jul 28, 2017 at 4:02 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Dave,

PFA patch to display additional information from pg_stat_activity table using subnode control.
RM#2597

Please review.

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


On Tue, Jul 25, 2017 at 10:56 AM, Shirley Wang <swang@pivotal.io> wrote:

On Mon, Jul 24, 2017 at 8:11 PM, Dave Page <dpage@pgadmin.org> wrote:


On Mon, Jul 24, 2017 at 3:28 PM, Shirley Wang <swang@pivotal.io> wrote:
2-3 days is a lot of valuable engineering time. Is this a 'drop everything now' kind of feature or can this wait for some user validation on a mock up first? 

Most of the time will likely be on the infrastructure to change the display to a subnode control. If you have some cycles to mockup potential layouts for the subnode view and have them validated, please feel free, however, that seems like an awful lot of work to me to display some missing SQL using a standard control.
Regarding SQL display: Developing simple control to show codemirror in disabled state (for now) wont take that much time.
 
Part of a product designer's job is to make sure there is a definitive need for a feature and that the interface for the feature is designed in such a way that the user gets all intended value from it. Time spent validating now will decrease the time spent later on redesigning / reimplementing.

If everyone is aware of what that value is and confident that how it'll be displayed is right, there's little risk in starting to develop it. If we're wrong, it'll add to feature bloat and detract from the experience.

Would Chethana be able to take on some of the design work? It would be valuable for the dev team to also be part of design process.





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

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



--
Chethana Kumar
Principal UI/UX Designer
EnterpriseDB Corporation


The Postgres Database Company




--
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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fixed issue where 'backend_type' column is existfor
Next
From: Harshal Dhumal
Date:
Subject: Re: pgAdmin 4 commit: Fix the feature tests failuers.