Re: [4696] Add RE-SQL/MSQL test cases for Materialized View

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [4696] Add RE-SQL/MSQL test cases for Materialized View
Дата
Msg-id CA+OCxoycmSXVQcEAjSNCf7Xu51mO75Z8UjStO4CNerzGuBcqBA@mail.gmail.com
обсуждение исходный текст
Ответ на [4696] Add RE-SQL/MSQL test cases for Materialized View  (Fahar Abbas <fahar.abbas@enterprisedb.com>)
Список pgadmin-hackers
Hi

On Thu, Sep 5, 2019 at 9:09 AM Fahar Abbas <fahar.abbas@enterprisedb.com> wrote:
Hi Hackers,

Please find attached the patch for RE-SQL for Materialized view.

Please review and apply the patch.

- Please be consistent with your naming - you have scenarios that mention both mview and MView. Any reason not to call them Materialised Views though?

- I would expect to see more alter steps; e.g. to change the name (thus ensuring that escaping is handled properly for that), to change autovac parameters, update the ACL etc. Whilst we probably cannot cover every possible scenario, we should at least try to get tests that cover all the basics.

- I honestly don't see how this is working (I need coffee maybe!). As far as I can see, the create and alter scenarios for both PG and EPAS use exactly the same parameters apart from the endpoints, yet the comment is being dropped and it's flipping to DATA from NO DATA. Are those actual bugs?

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

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

Предыдущее
От: Akshay Joshi
Дата:
Сообщение: Re: [4696] Add RE-SQL/MSQL test cases for Materialized View
Следующее
От: Akshay Joshi
Дата:
Сообщение: pgAdmin 4 commit: 1) Add Reverse Engineered and Modified SQL tests for