Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)

Поиск
Список
Период
Сортировка
От Khushboo Vashi
Тема Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)
Дата
Msg-id CAFOhELeK0uofjOTG-=WJQ9N5+GDxc4fvg+EqMJPyL3vBsbmuAw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Ответы Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
Hi,

I have fixed the issue and also reviewed the patch.
It looks good to me apart from the written RE-SQL test cases that don't test the SQL storage params (which will be added with the different RM as per our last conversation).

Thanks,
Khushboo

On Fri, Jul 10, 2020 at 5:31 PM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi Rahul,

I couldn't continue my review as I got an error on update.

Partitioned_sql_update.png

Thanks,
Khushboo

On Fri, Jul 10, 2020 at 5:19 PM Rahul Shirsat <rahul.shirsat@enterprisedb.com> wrote:
Hi Khushboo,

I had already corrected the patch as soon as I saw your mail, but couldn't send you as I went out on an urgency. Now sending it.

On Fri, Jul 10, 2020 at 10:25 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi Rahul,

I am getting errors while applying the patch, please rebase the patch and send it again.

Thanks,
Khushboo

On Thu, Jul 9, 2020 at 1:25 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Khushboo

Can you please review it.

On Wed, Jul 8, 2020 at 6:12 PM Rahul Shirsat <rahul.shirsat@enterprisedb.com> wrote:
Hi Hackers,

Following patch includes fix for : 

Storage parameters should be displayed in parent tables's SQL.

As the code will be upgraded in feature #5649, so only initial test cases for pg 10, 11 are included in this patch, so later it can be continued with the latest code.

--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.

On Mon, Jul 6, 2020 at 2:27 PM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi Rahul,

- The storage parameters should be displayed in the parent table's SQL too
- Please write test cases, preferably re-sql test cases

Thanks,
Khushboo

On Mon, Jul 6, 2020 at 11:28 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Khushboo,

Can you please review it.

On Sat, Jul 4, 2020 at 1:45 AM Rahul Shirsat <rahul.shirsat@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached patch which fixes the issue related to the partition table with storage parameters.

--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.


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


--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.


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


--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.
Вложения

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

Предыдущее
От: Khushboo Vashi
Дата:
Сообщение: Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)
Следующее
От: Dave Page
Дата:
Сообщение: RM5414 - Runtime paths