Re: Implementation of the GUI to create/modify/drop "Resource Group".

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: Implementation of the GUI to create/modify/drop "Resource Group".
Дата
Msg-id CA+OCxoxkk7+s+YQEFntrze9p9CL6ogT4ZT4EEfB1Rb_e1+OCaA@mail.gmail.com
обсуждение исходный текст
Ответ на Implementation of the GUI to create/modify/drop "Resource Group".  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Ответы Re: Implementation of the GUI to create/modify/drop "Resource Group".  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
Hi Akshay


On Thu, Jun 5, 2014 at 1:37 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Dave

I have implemented the GUI for creating, modifying and deleting the Resource Group for PPAS 9.4 and above. As I don't have new icon for the "Resource Group", I used the existing group and groups icon. I have tested the code on Windows and Linux.

Attached is the patch file. Please review it and if code looks good to you then can you please commit it.  

It looks fine to me on first review and test, except:

- The schema class and filename should be edbResourceGroup, not pgResourceGroup.

- pgResourceGroup::GetSql is incomplete - it doesn't emit the rate limit values, just "CREATE RESOURCE GROUP foo;" 

Thanks.

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

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

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

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: PATCH: Search Objects by definitions and comments
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin III commit: Soften up the collations icon.