Two things with that. You're setting the value of @PAMID, but not using it downstream. That means you're doing unnecessary reads against PAT_ACCOUNT_MANAGER. Second, your trigger will fail if you insert more than one row at a time, since your SET statements will attempt to get multiple values into a scalar variable.

John