This "eats the exception" - that's very bad, because the user doesn't know it went wrong.
Worse, it's eating any exception, for example including AuthenticationException.

Read this docs session, especially the note about the "fail fasting" philosophy:
  http://docs.jboss.org/drools/release/latest/optaplanner-docs/html_single/index.html#logging