Skip to content

Conversation

@sothawo
Copy link
Collaborator

@sothawo sothawo commented Jan 8, 2023

Closes #2419

@sothawo sothawo merged commit 44a5c75 into spring-projects:main Jan 8, 2023
sothawo added a commit that referenced this pull request Jan 8, 2023
Original Pull Request #2423
Closes #2419

(cherry picked from commit 44a5c75)
@patpatpat123
Copy link

Hello @sothawo

Thank you again for merging this.

I would like to help with the validation.

To avoid confusion, I am going to paste part my pom hoping it will contain your fix:

Should I go with:

<dependencies>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-data-elasticsearch</artifactId>
		</dependency>
<dependency>
  <groupId>org.springframework.data</groupId>
  <artifactId>spring-data-elasticsearch</artifactId>
  <version>5.1.0-SNAPSHOT</version>
</dependency>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-webflux</artifactId>
		</dependency>
	</dependencies>

i.e adding an extra block to spring-data-elasticsearch at version 5.1.0-SNAPSHOT?

please let me know as I am looking forward to validating this.

Thank you!

@sothawo
Copy link
Collaborator Author

sothawo commented Jan 9, 2023

When using Spring Data Elasticsearch 5.1.0-SNAPSHOT you should also add

<dependency>
  <groupId>org.springframework.data</groupId>
  <artifactId>spring-data-commons</artifactId>
  <version>3.1.0-SNAPSHOT</version>
</dependency>

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adapt AOT/native integration.

2 participants