Skip to content

AOT/Native support. #2423

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 8, 2023

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