Re: [pgAdmin4] [Patch]: Resource Group Module

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-9517Mobile: +91 976-788-8246*
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Akshay Joshi*
>>>>> *Principal Software Engineer *
>>>>>
>>>>>
>>>>>
>>>>> *Phone: +91 20-3058-9517Mobile: +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-9517Mobile: +91 976-788-8246*
>>>
>>
>>
>
>
> --
> *Akshay Joshi*
> *Principal Software Engineer *
>
>
>
> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>