Re: [PATCHES] Re: [HACKERS] Almost there on column aliases

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

In response to

Responses

Browse pgsql-hackers by date

  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 ...