You dont need this block `catch (final Exception e) {``

Also, if the oldValue is null you should simply return null after the warning message and not continue down. Agreed certainly that the logger should also use SLF4J. Also, I'd change the warn level to be error.