X-Original-To: pgsql-sql-postgresql.org@postgresql.org Received: from localhost (mx1.hub.org [200.46.208.251]) by postgresql.org (Postfix) with ESMTP id 2402F9FA42F for ; Fri, 21 Jul 2006 12:40:23 -0300 (ADT) Received: from postgresql.org ([200.46.204.71]) by localhost (mx1.hub.org [200.46.208.251]) (amavisd-new, port 10024) with ESMTP id 68929-02 for ; Fri, 21 Jul 2006 12:40:13 -0300 (ADT) X-Greylist: from auto-whitelisted by SQLgrey- Received: from sss.pgh.pa.us (sss.pgh.pa.us [66.207.139.130]) by postgresql.org (Postfix) with ESMTP id 6E7B59FA7BA for ; Fri, 21 Jul 2006 12:40:13 -0300 (ADT) Received: from sss2.sss.pgh.pa.us (tgl@localhost [127.0.0.1]) by sss.pgh.pa.us (8.13.6/8.13.6) with ESMTP id k6LFeBNY029388; Fri, 21 Jul 2006 11:40:11 -0400 (EDT) To: "Kevin Nikiforuk" cc: pgsql-sql@postgresql.org Subject: Re: Error when trying to use a FOR loop In-reply-to: <6EA72AC41F81D242A38AAF505523A94D022DF4@kappa.acrodex.net> References: <6EA72AC41F81D242A38AAF505523A94D022DF4@kappa.acrodex.net> Comments: In-reply-to "Kevin Nikiforuk" message dated "Fri, 21 Jul 2006 09:26:21 -0600" Date: Fri, 21 Jul 2006 11:40:11 -0400 Message-ID: <29387.1153496411@sss.pgh.pa.us> From: Tom Lane X-Virus-Scanned: Maia Mailguard 1.0.1 X-Spam-Status: No, hits=0.085 tagged_above=0 required=5 tests=AWL, SPF_HELO_PASS, SPF_PASS, UPPERCASE_25_50 X-Spam-Level: X-Archive-Number: 200607/277 X-Sequence-Number: 25851 "Kevin Nikiforuk" writes: > CREATE OR REPLACE FUNCTION rgio() RETURNS integer as $$ > BEGIN > DECLARE lv RECORD; > FOR lv IN SELECT DISTINCT rg FROM ldevrg LOOP > SELECT ldev FROM ldevrg WHERE ldevrg='$lv'; > END LOOP; > RETURN 1; > END; > $$ LANGUAGE plpgsql; You've got a small error in the layout: the DECLARE part goes before BEGIN not after. Swap the first two lines of the function. regards, tom lane