search

dd4t / dd4t-2-java

10 stars 21 forks source link

Lower some log levels #101

Closed Dretch closed 6 years ago

Dretch commented 7 years ago

Does creating a new instance of JsonDataBinder really justify an info log message? Personally I don't think it does - hence the first change.

The second change reduces a warn to a debug, because a warning should indicate some kind of problem, but this message happens even when the view models are setup correctly (I have @ViewModel(rootElementNames = "myElement"), so I get a redundant warning like Could not load Model Class for key: my-element).

RaimondKempees commented 7 years ago

We may as well remove that constructor log line. I added this ages ago to test the singletonness, but since we're now using Spring, it makes no sense. What do you think?

Dretch commented 7 years ago

I agree - I removed the log line completely.