Added to TODO under PL/pgSQL:

                o Allow row and record variables to be set to NULL constants,
                  and allow NULL tests on such variables

                  Because a row is not scalar, do not allow assignment
                  from NULL-valued scalars.


---------------------------------------------------------------------------

Jim C. Nasby wrote:
> On Tue, Feb 13, 2007 at 05:55:11PM -0500, Bruce Momjian wrote:
> > 
> > Is there a TODO here?
> > 
> > ---------------------------------------------------------------------------
> > 
> > Jim Nasby wrote:
> > > On Oct 2, 2006, at 6:28 PM, Tom Lane wrote:
> > > > "Jim C. Nasby" <[EMAIL PROTECTED]> writes:
> > > >> However, the test right above that means that we'll fail if the user
> > > >> tries something like "row_variable := NULL;":
> > > >
> > > > The patch you seem to have in mind would allow
> > > >         row_variable := int_variable;
> > > > to succeed if the int_variable chanced to contain NULL, which is  
> > > > surely
> > > > not very desirable.
> 
> Well, that's Tom's objection, though I'm not sure if by 'int_variable'
> he means 'internal' or 'integer'.
> 
> Personally, I think it would be useful to just allow setting a row or
> record variable to NULL as I showed it above; ie: no variables involved.
> This is something you might want to do to invalidate a row/record
> variable after taking some action (perhaps deleting a row).
> 
> You'd also think that you should be able to detect if a record variable
> is null, as you can with row.
> 
> So, I suggest:
> 
> * Allow row and record variables in plpgsql to be set to NULL
> 
>     It's not clear if it's a wise idea to allow this assignment from a
>     variable. It may be better to only allow explicitly setting them,
>     ie:
> 
>     row_variable := NULL;
> 
> * Allow testing a record variable to see if it's NULL
> 
>     Currently works for row variables, but not record variables
> -- 
> Jim Nasby                                            [EMAIL PROTECTED]
> EnterpriseDB      http://enterprisedb.com      512.569.9461 (cell)

-- 
  Bruce Momjian  <[EMAIL PROTECTED]>          http://momjian.us
  EnterpriseDB                               http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster

Reply via email to