Re: pgAdmin III: the debugger does not pre-populate default values for arguments

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Nikhil S <nixmisc(at)gmail(dot)com>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: pgAdmin III: the debugger does not pre-populate default values for arguments
Date: 2011-02-11 16:27:23
Message-ID: AANLkTikAJL0Jzg2knNSOXm49x2-jZc7YOrtdQfF32SHs@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Fri, Feb 11, 2011 at 5:11 PM, Nikhil S <nixmisc(at)gmail(dot)com> wrote:
>>
>>
>> > Note that if there are no default values, the the defvals column is
>> > completely empty. Now why we see the "" for non-default arguments - the
>> > issue seems to be on the server side in the contrib module calls to
>> > compute
>> > the default values.
>>
>> We need to change that then - we can't have the missing default values
>> default to something that's not valid for that datatype. Change them
>> to null or just empty them?
>>
>
> Hmmm, well variables can default to NULL too. The backend code tries to
> track those variables which default to NULL or to other values. For the
> rest, it defaults them to "". It then uses construct_md_array to generate
> the output string for type TEXTOID. Dunno if we can have nothing (an empty
> string) as valid TEXTOID output. I guess it's a limitation for this type.
> Let me see if we can do something here on the backend side...

Doesn't really help, as we need to ensure it works sanely with
existing servers too.

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

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2011-02-11 16:56:09 Re: phase 2 of wxWidgets 2.9 build
Previous Message Nikhil S 2011-02-11 16:11:33 Re: pgAdmin III: the debugger does not pre-populate default values for arguments