Skip to content

Conversation

@filiphr
Copy link
Contributor

@filiphr filiphr commented Nov 15, 2025

When I did #47945 I was not aware that there was a new spring-boot-starter-elasticsearch in Spring Boot 4. @anbusampath brought this to my attention with #47945 (comment).

I'll leave this up to the Spring Boot team to decide whether the starter should be more opinionated and bring in the High Level elasticsearch-java client. In my opinion I believe that spring-boot-elasticsearch should stay as it is, i.e. only bring in the minimal dependencies, the elasticsearch-rest5-client. However, the starter can be a bit more opinionated and bring in more.

@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 15, 2025
@wilkinsona wilkinsona added this to the 4.0.x milestone Nov 15, 2025
@snicoll snicoll self-assigned this Nov 15, 2025
snicoll pushed a commit that referenced this pull request Nov 15, 2025
@snicoll snicoll closed this in a92cb95 Nov 15, 2025
@snicoll
Copy link
Member

snicoll commented Nov 15, 2025

Thanks @filiphr - I've removed the exclusion for commons-logging as we don't need to do that anymore. See also #48142.

@snicoll snicoll modified the milestones: 4.0.x, 4.0.0 Nov 15, 2025
@filiphr filiphr deleted the elasticsearch-java branch November 15, 2025 10:29
@filiphr
Copy link
Contributor Author

filiphr commented Nov 15, 2025

Thanks a lot @snicoll. Thanks for adjusting the docs as well, I forgot about them

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants