From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
Cc: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4] [Patch]: Resource Group Module |
Date: | 2016-02-22 12:00:34 |
Message-ID: | CA+OCxowGX5iEM9-eZ-aGnLPu6h01zaddZ+7d7ZaW1Ubb7pNi+g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Committed, with a couple of additional minor changes:
- s/resourcegroups/resource_groups (for consistency with server_groups)
- Added a PyDoc module description to __init__.py
Thanks!
On Mon, Feb 22, 2016 at 4:29 AM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
wrote:
> Hi Akshay,
>
> After implementation of the NumericControl, ResourceGroup patch looks good
> to me.
> As resource group does not have any dependencies on "Dependent" and
> "Dependencies" tabs so we can commit this patch.
>
> Thanks,
> Neel Patel
>
> On Tue, Feb 16, 2016 at 11:56 AM, Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Hi All
>>
>> On Mon, Feb 15, 2016 at 12:05 PM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com
>> > wrote:
>>
>>> Hi Akshay,
>>>
>>> Please find below review comments.
>>>
>>> - We are getting error at python side saying "*AttributeError:
>>> 'Response' object has no attribute 'strip'*"
>>> - "CPU Rate Limit" and "Dirty Rate Limit" should not allowed to
>>> enter the character (a-z). Currently it allows to enter invalid character
>>> so we are getting below SQL error saying "*LINE 2: SET
>>> cpu_rate_limit = 4545, dirty_rate_limit = gfgf;*"
>>>
>>> Both above errors are due to not validated "CPU Rate Limit" and "Dirty
>>> Rate Limit" fields.
>>>
>>
>> To fixed the above I have implemented NumericControl with validation.
>> I have already send separate patch for that. Attached is the new patch
>> which contains the above fix, documentation and fix for "Refresh Properties
>> tab after adding/updating the node". Before applying this patch, you need
>> to apply the NumericControl patch. Please review it.
>>
>>>
>>> Thanks,
>>> Neel Patel
>>>
>>> On Mon, Feb 8, 2016 at 4:13 PM, Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi All
>>>>
>>>> Attached is the patch with the fixes. Please review it and if looks
>>>> good then please commit it.
>>>>
>>>> On Mon, Feb 8, 2016 at 11:54 AM, Neel Patel <
>>>> neel(dot)patel(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Akshay,
>>>>>
>>>>> Please find below review comments.
>>>>>
>>>>> - As per the pgAdmin3, we should allow to enter the floating point
>>>>> value in "CPU Rate Limit" and "Dirty Rate Limit".
>>>>>
>>>> Fixed.
>>>>
>>>> - While creating resource group, we are getting error saying "*generate_browser_node()
>>>>> missing 1 required positional argument: 'label'*".
>>>>>
>>>> Fixed.
>>>>
>>>> - Valid range for CPU Rate Limit and Dirty Rate Limit is from 0 to
>>>>> 1.67772e+07. We are not able to enter the maximum limit value while
>>>>> creating the
>>>>> resource group.
>>>>>
>>>>
>>>> Partially fixed, I have limit the max length to 8 digits, but not
>>>> able to fixed it to the exact range.
>>>>
>>>>>
>>>>> Thanks,
>>>>> Neel Patel
>>>>>
>>>>> On Fri, Feb 5, 2016 at 6:15 PM, Akshay Joshi <
>>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi All
>>>>>>
>>>>>> I have added logic to show sql in the "SQL" tab on dashboard and also
>>>>>> fixed some warnings. Please refer and review this patch.
>>>>>>
>>>>>> On Thu, Feb 4, 2016 at 5:50 PM, Akshay Joshi <
>>>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>>>
>>>>>>> Hi All
>>>>>>>
>>>>>>> Attached is the patch for the Resource Group Module. Please review
>>>>>>> it and if it looks good then please commit it.
>>>>>>>
>>>>>>> --
>>>>>>> *Akshay Joshi*
>>>>>>> *Principal Software Engineer *
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> *Phone: +91 20-3058-9517 <%2B91%2020-3058-9517>Mobile: +91
>>>>>>> 976-788-8246*
>>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> *Akshay Joshi*
>>>>>> *Principal Software Engineer *
>>>>>>
>>>>>>
>>>>>>
>>>>>> *Phone: +91 20-3058-9517 <%2B91%2020-3058-9517>Mobile: +91
>>>>>> 976-788-8246*
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org
>>>>>> )
>>>>>> To make changes to your subscription:
>>>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>>>
>>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> *Akshay Joshi*
>>>> *Principal Software Engineer *
>>>>
>>>>
>>>>
>>>> *Phone: +91 20-3058-9517 <%2B91%2020-3058-9517>Mobile: +91 976-788-8246*
>>>>
>>>
>>>
>>
>>
>> --
>> *Akshay Joshi*
>> *Principal Software Engineer *
>>
>>
>>
>> *Phone: +91 20-3058-9517 <%2B91%2020-3058-9517>Mobile: +91 976-788-8246*
>>
>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2016-02-22 12:06:40 | pgAdmin 4 commit: public in the privilege control should not be allowed |
Previous Message | Dave Page | 2016-02-22 11:59:51 | Re: [pgAdmin4] [Patch]: Language Module |