G-7810: Never use SQL inside PL/SQL to read sequence numbers (or SYSDATE).

Major

Efficiency, Maintainability

Reason

Since Oracle 11g it is no longer needed to use a select statement to read a sequence (which would imply a context switch).

Example (bad)

declare
   l_sequence_number employees.employee_id%type;
begin
   select employees_seq.nextval
     into l_sequence_number
     from dual;
end;
/

Example (good)

declare
   l_sequence_number employees.employee_id%type;
begin
   l_sequence_number := employees_seq.nextval;
end;
/