fix maven elasticsearch missing dependencies

[ruddell]

Maven projects using Elasticsearch fail to start without these dependencies.

Pinging @Tcharl since they may know a better way to fix this (these dependencies were removed in #8475). I don't notice anything wrong when adding these dependencies back to the pom.xml.

related to #8739

• Please make sure the below checklist is followed for Pull Requests.

• Travis tests are green

• Tests are added where necessary

• Documentation is added/updated where necessary

• Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

[jdubois]

Sorry I missed this - should I do an emergency patch request tomorrow?

[ruddell]

If you have time it would be good

[jdubois]

Thanks! I'll try tomorrow

[DanielFran]

@ruddel Is this needed to explicitly define the version?

[ruddell]

@DanielFran It fails in dev if I remove the specific versions:
{"root_cause":[{"type":"class_cast_exception","reason":"org.apache.logging.slf4j.SLF4JLoggerContext cannot be cast to org.apache.logging.log4j.core.LoggerContext"}],"type":"class_cast_exception","reason":"org.apache.logging.slf4j.SLF4JLoggerContext cannot be cast to org.apache.logging.log4j.core.LoggerContext"}

[DanielFran]

From @Tcharl 's PR, he added the mapstruct + log4j dependencies in jhipster-bom project but then remove it since it was failing... He probably forget to rollback changes on generator-jhipster project.

And current logj2 version is 2.10.0 with spring-boot 2.0.6.
Also slf4j version 1.7.25.

[DanielFran]

Probably missing the bidding between slf4j to log4j dependency: https://logging.apache.org/log4j/2.0/log4j-slf4j-impl/

Validated the starter-data-jest and the log4j dependency is only defined for testing... Seems an issue with dependencies!
@VanRoy maybe related with VanRoy/spring-data-jest#90 and VanRoy/spring-data-jest#95

[pascalgrimaud]

Do we have the issue with Gradle too ?
@ruddell : is it only for dev profile, right ? That explains why our CI didn't detect it

[ruddell]

The dependencies are still in the gradle file here

And yes it only affects dev

[jdubois]

Hi everyone, I understand this isn't a perfect solution, but let's first fix this and then try to improve. I'm merging this and I'll try to do a release today.