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

From: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)
Date: 2020-07-10 04:55:48
Message-ID: CAFOhELchih0Qi_Hb01-SnipHjpvwx2X3PTT+K5b1fzhnYGY+5A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Hi Khushboo
>
> Can you please review it.
>
> On Wed, Jul 8, 2020 at 6:12 PM Rahul Shirsat <
> rahul(dot)shirsat(at)enterprisedb(dot)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
>> <https://redmine.postgresql.org/issues/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(dot)vashi(at)enterprisedb(dot)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(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Khushboo,
>>>>
>>>> Can you please review it.
>>>>
>>>> On Sat, Jul 4, 2020 at 1:45 AM Rahul Shirsat <
>>>> rahul(dot)shirsat(at)enterprisedb(dot)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*
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Yogesh Jain 2020-07-10 05:29:12 [pgAdmin] Regarding Alert-box on login page
Previous Message Nikhil Mohite 2020-07-09 13:48:25 [pgAdmin][RM5329][Code Coverage] Improve API test cases for FTS Configuration, FTS Dictionary, FTS Parser, FTS Template