From: | Thomas Lockhart <lockhart(at)alumni(dot)caltech(dot)edu> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Postgres Hackers List <hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCHES] Re: [HACKERS] Almost there on column aliases |
Date: | 2000-02-15 03:21:01 |
Message-ID: | 38A8C61D.90557A4B@alumni.caltech.edu |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> > Right. I'm looking forward to advice on the right way to do this. The
> > problem is that the introductory character for list structures is
> > *also* the introductory character for plans, so everything blows
> > chunks if I just call nodeRead() from _readAttr().
> Huh? '{' introduces a node, '(' introduces a list. See the comments
> I added (not very long ago :-() in read.c. My guess is that you are
> either emitting the wrong character or have some sort of error in the
> way you call nodeRead. Nothing obviously wrong in the patch diffs
> though.
The problem I recall is that paren also introduces a "plan", and if
you call nodeRead() it sees the paren and then complains later because
it expects a node label following the paren.
I probably misdiagnosed the behavior, but in any case I'd be *really*
happy if someone wants to put me out of my misery on this one ;)
- Thomas
--
Thomas Lockhart lockhart(at)alumni(dot)caltech(dot)edu
South Pasadena, California
From | Date | Subject | |
---|---|---|---|
Next Message | Chris Bitmead | 2000-02-15 03:25:41 | Re: [HACKERS] Solution for LIMIT cost estimation |
Previous Message | Alfred Perlstein | 2000-02-15 02:55:24 | Re: [HACKERS] pid_t define missing in include/miscadmin.h ... |