MIME-Version: 1.0
In-Reply-To: 
 <CACCA4P1Rm1Z1em+XMO5qy3Ne=ckyMMbUeffsNKPWti1z7Abziw@mail.gmail.com>
References: <E1ZlSve-0007yw-0c@gothos.postgresql.org>
 <568BA29D.2070305@yandex.ru>
 <CA+OCxoz4ZaoKk7p77H6qZ-CpFVnTyxVj9uLxar5Fqge3Tmeg1g@mail.gmail.com>
 <CACCA4P1_OuwX5RLcQK==FEG5dL8mUt=r56MwNGyMfb3G8cqODw@mail.gmail.com>
 <CACCA4P1Rm1Z1em+XMO5qy3Ne=ckyMMbUeffsNKPWti1z7Abziw@mail.gmail.com>
Date: Wed, 6 Jan 2016 09:59:13 +0000
Message-ID: 
 <CA+OCxowi3=kz9-91UiVjBvQFndXePhOsYT9KznF_GPn5vTeB8A@mail.gmail.com>
Subject: Re: pgAdmin III commit: As per
 commit:600daf650bd5e50e664c4ece2e53e95446390
From: Dave Page <dpage@pgadmin.org>
To: Neel Patel <neel.patel@enterprisedb.com>
Cc: Nikolai Zhubr <n-a-zhubr@yandex.ru>,
 pgadmin-hackers <pgadmin-hackers@postgresql.org>,
 Ashesh Vashi <ashesh.vashi@enterprisedb.com>
Content-Type: text/plain; charset=UTF-8
Precedence: bulk
Sender: pgadmin-hackers-owner@postgresql.org

Thanks. Ashesh - can you review/commit please?


On Tue, Jan 5, 2016 at 12:24 PM, Neel Patel <neel.patel@enterprisedb.com> wrote:
> Hi,
>
> Please find attached patch file with below fix.
>  - Return value of the function will be displayed as "SETOF record" instead
> of "SETOF SETOF record" under SQL tab within properties window.
>
> Let me know for any comments.
>
> Thanks,
> Neel Patel
>
> On Tue, Jan 5, 2016 at 5:08 PM, Neel Patel <neel.patel@enterprisedb.com>
> wrote:
>>
>> Hi Dave,
>>
>> Sure, I will look into this issue.
>>
>> Thanks,
>> Neel Patel
>>
>> On Tue, Jan 5, 2016 at 4:47 PM, Dave Page <dpage@pgadmin.org> wrote:
>>>
>>> Neel, can you look at this please?
>>>
>>> Thanks.
>>>
>>> On Tue, Jan 5, 2016 at 11:01 AM, Nikolai Zhubr <n-a-zhubr@yandex.ru>
>>> wrote:
>>> > Hi all,
>>> >
>>> > Running 1.22-rc1 now I observe that this commit has apparently still
>>> > not
>>> > solved the issue in at least one place. That is:
>>> >
>>> > - "RETURNS SETOF record" is correctly displayed in SQL panel now.
>>> >
>>> > - "RETURNS SETOF SETOF record" is still erroneously displayed under SQL
>>> > tab
>>> > within the properties window (effectively preventing set-returning
>>> > function
>>> > to be changed in any way through the properties window).
>>> >
>>> >
>>> > Thank you,
>>> > Nikolai
>>> >
>>> > 12.10.2015 5:33, Ashesh Vashi:
>>> >>
>>> >> As per commit:600daf650bd5e50e664c4ece2e53e95446390b6d, we now use
>>> >> pg_get_function_result(..) function to identify the return type of the
>>> >> function, hence - we don't need to add SETOF keyword explicitly, when
>>> >> a
>>> >> function returns set of records.
>>> >>
>>> >> Branch
>>> >> ------
>>> >> master
>>> >>
>>> >> Details
>>> >> -------
>>> >>
>>> >>
>>> >> http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=9d7596dc552cf9315d4aa83eafe19db1acf7e531
>>> >>
>>> >> Modified Files
>>> >> --------------
>>> >> pgadmin/schema/pgFunction.cpp |    2 --
>>> >> 1 file changed, 2 deletions(-)
>>> >>
>>> >>
>>> >
>>> >
>>> >
>>> > --
>>> > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
>>> > To make changes to your subscription:
>>> > http://www.postgresql.org/mailpref/pgadmin-hackers
>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>
>>
>


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

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


-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers