Какие типы у данного поля и переменной?

Это ж как надо надругаться над таблицей, чтоб в decimal и numeric числа САМИ СОБОЙ округлялись...
>вместо переменных использовал просто update
ымгм. прямо и сказать-то нечего

SergSuper, Вы не в духе? Обычно Ваше появление на форуме приносит ценную и эмоционально нейтральную информацию. Сегодня что-то не так. Улыбнитесь

Если генерация sf_id укладывается в функцию, то весь этот огород с курсорами, инсертами и апдейтами можно заменить 2-мя строчками:

insert into table2 (sf_id, shifr, sum_uslug)
select gen_sf_id(), shifr, sum_uslug from table1

Если забыть о деталях генерации (вы же не сразу написали, в чем состоит генерация sf_id), то принцип до безобразия прост: создаем функцию, обзывая ее, к примеру, gen_sf_id()

create function gen_sf_id()
returns int
as
begin
-- generate some value with some rules
return (select max(sf_id) + 1 from table2)
end

а потом вся вставка НАБОРА значений из одной таблицы в другую делается ОДНИМ оператором:

insert into table2 (sf_id, shifr, sum_uslug)
select gen_sf_id(), shifr, sum_uslug from table1

Эта штука заменяет весь цикл по курсору.
----------------------------------------
Теперь. Если все-таки обратить внимание, на то, что вы делаете ПО СМЫСЛУ, то все это бодяга здорово настораживает. Почему не использовать identity? Зачем эти самоделки насчет max(sf_id) + 1? А вы продумали работу системы при множестве пользователей? Нет. Вам не пришло в голову, что между запросом максимального значения и вставкой его плюс 1 в другую таблицу есть разница во времени? Если в этот промежуток другой пользователь выполнит этот код, то будет вставка ДВУХ строк с одним и тем же sf_id. Да и при увеличении таблицы поиск максимального значения будет все более и более накладной операцией. Нужны еще доводы или этих хватит?