Re: [pgAdmin4][Patch]: RM 5053 - Getting an error while changing thecolumns in the existing view

Поиск
Список
Период
Сортировка
От Akshay Joshi
Тема Re: [pgAdmin4][Patch]: RM 5053 - Getting an error while changing thecolumns in the existing view
Дата
Msg-id CANxoLDdfJGOASsQVknwHkbk+L4zWP5UdWjPZBsVqtg8hL6-SfA@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin4][Patch]: RM 5053 - Getting an error while changing thecolumns in the existing view  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Ответы Re: [pgAdmin4][Patch]: RM 5053 - Getting an error while changing thecolumns in the existing view  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Список pgadmin-hackers
Дерево обсуждения
Hi Khushboo

Following are the review comments:
  • Fix the PEP8 issue.
  • Drop query should be part of the jinja template for consistency. Currently, it is added through the python file.
  • Any changes in the view code should not warn the user "Changing the columns in a view requires dropping...." and we should not drop the view. For example, I have only change the WHERE clause or added 'ORDER BY'. 


On Tue, Jan 14, 2020 at 10:27 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch for RM #5053 - Getting an error while changing the columns in the existing view.

PostgreSQL doesn't allow to change the view columns. So, while performing this task the existing view should be dropped first and then recreate it and also user will get a warning first.

Thanks,
Khushboo


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246

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

Предыдущее
От: Khushboo Vashi
Дата:
Сообщение: [pgAdmin4][Patch]: RM 5053 - Getting an error while changing thecolumns in the existing view
Следующее
От: Yogesh Mahajan
Дата:
Сообщение: RM5049 Patch