Pobierz wartość kolumny poprzedniego wiersza w postgresie nie może użyć funkcji okna w UPDATE

Zakładając, że ...

• gwma_duration i duration powinny być w tej samej kolumnie i różnić się literówkami.

• Chcesz zamówić według kolumny o nazwie order_column . Zastąp swoją rzeczywistą kolumną (kolumnami).

• Twoje kolumny klucza podstawowego to res_id . Zastąp swoją rzeczywistą kolumną (kolumnami).

Połóż szminkę na świni:

Twój kod proceduralny został naprawiony i ulepszony:

CREATE OR REPLACE FUNCTION vin_calc()
  RETURNS void AS
$func$
DECLARE
   r res%rowtype;
   i integer := 0;
   last_grp text;
BEGIN

FOR r IN
   SELECT * FROM res
LOOP
   IF last_grp <> r.prod_grp_nm THEN
      i := 1;
   ELSE
      i := i + 1;
   END IF;

   IF i < 3 THEN
      UPDATE res
      SET    duration = i - 1
      WHERE  dur = r.dur
      AND    prod_grp_nm = r.prod_grp_nm
      AND    week_end = r.week_end;

   ELSE
      UPDATE res r1
      SET    duration = r.dur * 0.125 + 
            (SELECT 0.875 * gwma_duration FROM res
             WHERE order_column < r1.order_column
             ORDER BY order_column
             LIMIT 1
            )  -- could be replaced with last_duration, analog to last_grp
      WHERE  r1.dur = r.dur
      AND    r1.prod_grp_nm = r.prod_grp_nm
      AND    r1.week_end = r.week_end;
   END IF;

   last_grp := r.prod_grp_nm;

   END LOOP;
END
$func$
LANGUAGE plpgsql;

• Użyj niejawnego kursora FOR pętla . Nie ma potrzeby używania nieporęcznego, wyraźnego kursora.

• Nigdy nie cytuj nazwy języka plpgsql , który jest identyfikatorem, a nie ciągiem.

• Uprościliśmy logikę w kilku miejscach.

• Co najważniejsze , jak mówi komunikat o błędzie, nie możesz używać funkcji okna w SET klauzula UPDATE . Zastąpiłem go skorelowanym podzapytaniem. Ale prawdopodobnie można by go zastąpić przez last_duration , analogicznie do last_grp :po prostu zapamiętaj wartość z ostatniej iteracji.

Właściwe rozwiązanie

Jednak wszystkie powyższe są bardzo nieefektywne, gdy możesz to zrobić w pojedynczej UPDATE oświadczenie :

UPDATE res r
SET    duration = CASE WHEN r0.rn < 3
                     THEN r0.rn - 1
                     ELSE r0.last_dur * 0.875 + r.dur * 0.125
                  END
FROM  (
   SELECT res_id, duration
        , row_number()  OVER (PARTITION BY prod_grp_nm ORDER BY order_column) AS rn
        , lag(duration) OVER (PARTITION BY prod_grp_nm ORDER BY order_column) AS last_dur
   FROM res
   ) r0
WHERE  r.res_id = r0.res_id

• Żeby było jasne:możesz użyj funkcji okna w FROM klauzula — przynajmniej we współczesnych wersjach Postgresa.

• Użyj row_number() , a nie rank() odpowiadać Twojemu kodowi proceduralnemu.