Re: [pgAdmin4][RM#3014] Fix validation issues while creating new sequence

Поиск
Список
Период
Сортировка
От Murtuza Zabuawala
Тема Re: [pgAdmin4][RM#3014] Fix validation issues while creating new sequence
Дата
Msg-id CAKKotZQKOMb3Lwv3vCEEf7ux2xwm7=-jXb-428CVn56W8oD1Qg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][RM#3014] Fix validation issues while creating new sequence  (Joao De Almeida Pereira <jdealmeidapereira@pivotal.io>)
Ответы Re: [pgAdmin4][RM#3014] Fix validation issues while creating new sequence
Список pgadmin-hackers
Дерево обсуждения
Hi Joao,

​Thank you for your time in reviewing the patch.​


On Wed, Feb 21, 2018 at 9:24 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hi Murtuza,
Looks good. 
Was there any reason for the change in position of the if statements in the sql file. Makes the output more readable?
​Yes​
 
Looks like we need some test to ensure that the Sequence issue does not happen again. Maybe a Unit Test is good enough, to ensure that we are creating a good SQL
​I'll add a test case for the same.​
 

Thanks
Joao

On Wed, Feb 21, 2018 at 3:13 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patches,
1) To fix the validation issue while creating new sequence
RM#3014

2) Fix PEP-8 issues in sequence module
pycodestyle --config=.pycodestyle ./pgadmin/browser/server_groups/servers/databases/schemas/sequences/

3) To fix the regression test for Tablespace when running with Python3.x
RM#3138

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


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