The static final logger fields should follow the naming conventions for constants. I wouldn't bother doing it explicitly, but I'm taking it as an example for exploring automated refactorings using Netflix Rewrite.