Re: [pgAdmin4] [Patch]: Resource Group Module
От | Neel Patel |
---|---|
Тема | Re: [pgAdmin4] [Patch]: Resource Group Module |
Дата | |
Msg-id | CACCA4P00-CGnfFwYvFgrfet6UdzDbDHApBRPpWcbKuNg6YCT5g@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [pgAdmin4] [Patch]: Resource Group Module (Akshay Joshi <akshay.joshi@enterprisedb.com>) |
Ответы |
Re: [pgAdmin4] [Patch]: Resource Group Module
|
Список | pgadmin-hackers |
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.joshi@enterprisedb.com> wrote:
Hi AllOn Mon, Feb 15, 2016 at 12:05 PM, Neel Patel <neel.patel@enterprisedb.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 PatelOn Mon, Feb 8, 2016 at 4:13 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi AllAttached 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.patel@enterprisedb.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 theresource group.Partially fixed, I have limit the max length to 8 digits, but not able to fixed it to the exact range.Thanks,Neel PatelOn Fri, Feb 5, 2016 at 6:15 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi AllI 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.joshi@enterprisedb.com> wrote:Hi AllAttached is the patch for the Resource Group Module. Please review it and if it looks good then please commit it.--Akshay JoshiPrincipal Software EngineerPhone: +91 20-3058-9517
Mobile: +91 976-788-8246--Akshay JoshiPrincipal Software EngineerPhone: +91 20-3058-9517
Mobile: +91 976-788-8246
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers--Akshay JoshiPrincipal Software EngineerPhone: +91 20-3058-9517
Mobile: +91 976-788-8246--Akshay JoshiPrincipal Software EngineerPhone: +91 20-3058-9517
Mobile: +91 976-788-8246
В списке pgadmin-hackers по дате отправления:
Предыдущее
От: Akshay JoshiДата:
Сообщение: Re: [pgAdmin4][Patch]: Dependecies/Dependents functionality
Следующее
От: Ashesh VashiДата:
Сообщение: pgAdmin 4 commit: Resolved few issues (with some improvements) with exi