try {...}
catch (Exception ex) {
LOGGER.error("updateDropShipPriceOnImport()", ex);
throw ex;
}
Now, why would you log the error only to rethrow it? I'd assume that if you're rethrowing it, something else is/should catch this and handle/log it. So then isn't this just unnecessary logging?