Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard

Поиск
Список
Период
Сортировка
От Murtuza Zabuawala
Тема Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard
Дата
Msg-id CAKKotZTAz8hBRk=nAC6nJkS7PawmA25ZJP5uUjBCoBkpHk8CRw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard  (Dave Page <dpage@pgadmin.org>)
Ответы Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard  (Shirley Wang <swang@pivotal.io>)
Список pgadmin-hackers

On Mon, Jul 17, 2017 at 3:01 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Jul 13, 2017 at 2:53 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
hiHi,

PFA patch to add functionality which will allow super user to cancel long running queries from dashboard.
RM#1812

Steps used to test:
1) Open psql session, Connect to 'test' database on respective server
2) Execute "select pg_sleep(1000);"
3) Open pgAdmin4
4) Connect to respective server
5) Click on Dashboard
6) Check "Sessions" tab under "Server activity" section then look for active sessions for test database.
7) Click on cancel button and cancel the active session
8) Check psql session now, you will see "ERROR:  canceling statement due to user request"

Some comments:

- The action here is to cancel the active query in the backend, not the session - so messages etc. should say things like "Cancel Active Query?"

- The grid should refresh following the cancel operation.

- Can you fix the vertical alignment while you're working on this? The new button really makes the poor alignment stand out.

- This should not be superuser only - regular users should be able to cancel their own queries.

- On PG10, background workers are also shown in the dashboard. Should we prevent attempts to cancel their work (they'll fail anyway I believe). 

Thanks!
 
Sure, I'll work on these comments & send updated patch. 
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][Patch]: Allow user to Comment/Uncomment code in query editor
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin 4 commit: Stop tool buttons from changing their stylingunexpec