您在这里遇到一些问题,包括:
IN_DATE
被声明为日期,因此您无需将其传递给TO_DATE()
。- 您只需要一个游标循环;如果
employee_id
出于某种原因要一起处理所有更新,则可以添加一个order by
子句。 - 您根本不需要动态SQL。您可以将游标中的值用作静态SQL更新的一部分。
因此,具有单个循环的简单版本可能类似于:
CREATE OR REPLACe PROCEDURE sp_run_employee_updates (p_date IN DATE) IS CURSOR c_updates IS SELECT * FROM bi_employee_update WHERe effective_date = p_date AND executed = 'N' AND activity_id = '0' FOR UPDATE; BEGIN -- loop around all pending records FOR r_update IN c_updates LOOP -- apply this update to the bi_employee record UPDATE bi_employee SET col1 = r_update.col1, col2 = r_update.col2 WHERe emp_id = r_update.employee_id; -- mark this update as executed UPDATE bi_employee_update SET executed = 'Y' WHERe CURRENT OF c_updates; END LOOP;END sp_run_employee_updates;
这是使用
for updateand
where currentof构造来锁定您正在使用的行并简化更新;请参阅此处的文档。
值得一提的是,如果其中一个
effective_date或
p_date一个没有时间部分,它们将不匹配。这不太可能
p_date,但更难猜测
effective_date。如果确实如此,那么您要么需要
trunc()它,要么用它
between寻找一段时间。



