Re: I'm stuck - I just can't get this small FUNCT to run!

From: Ralph Smith <rsmith(at)10kinfo(dot)com>
To:
Cc: pgsql-sql(at)postgresql(dot)org
Subject: Re: I'm stuck - I just can't get this small FUNCT to run!
Date: 2010-11-04 20:07:02
Message-ID: 4CD31266.2000506@10kinfo.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-sql

Thank you both. I stared and stared and stared, and apparently I needed
to take a walk around the building
I have repeatedly asked about an upgrade, but as you mentioned Tom, it's
a compatibility issue and concern that remote programmers might end out
making more work for themselves. <<shrug>>

If I did a local install of PG 8.X (which is what I hear we'll be going
to), not connected to the real DB, would it be functional enough to give
me the better messages?

Again, thank you both,
Ralph
---------------------------------------------------------------

bricklen wrote:

> You appear to be missing a trailing semi-colons.
> Here:
> vFieldName= ''offer_'' || ''$1''
> and here: vBusID=daRec.bus_id
-------------------------------------------------------

Ralph Smith <rsmith(at)10kinfo(dot)com> writes:

> > I'm also stuck on 7.4 for at least a few more months, but that's not
> > part of the problem.
>

... well, actually, it is; because newer versions give considerably more
helpful syntax error messages. I tried plugging this function into
a current version, and got

syntax error at or near "daRec"
LINE 34: FOR daRec IN SELECT * FROM temp_salesimport WHERE offer_...
^

which was at least close enough to the problem (the missing semi on the
previous line) to be of some use, unlike the error pointing at line 92.
Similarly, after fixing the first missing semi I got

ERROR: syntax error at or near "select"
LINE 38: select into vBusOwnerID businessownerid from business ...
^

which is just after the other one.

If you aren't in a position to move your production DB yet, you might
at least consider using a newer version for development. That has its
own hazards of course, like accidentally using features that don't exist
in 7.4, but it could save you a lot of time in cases like this.

regards, tom lane

--

Ralph
_________________________

In response to

Browse pgsql-sql by date

  From Date Subject
Next Message Will Furnass 2010-11-07 10:18:05 Re: resolved: WITH RECURSIVE: ARRAY[id] All column datatypes must be hashable
Previous Message Kenneth Marshall 2010-11-04 19:47:46 Re: Trailing spaces - is there an easier way?