Skip to content

Conversation

masseyke
Copy link
Member

@masseyke masseyke commented Feb 3, 2022

This commit makes it so that es-hadoop does not blow up when it is packaged within a Spring Boot application.
Closes #1359

@masseyke
Copy link
Member Author

masseyke commented Feb 3, 2022

I cloned the project at https://github.com/maxmin93/agens-spark to test this. I realized that the problem is more than just that it's a jar with in a jar -- it is also complicated by the fact that Spring Boot provides its own custom URLStreamHandler. That URLStreamHandler results in the URLConnection returning a different jar file URL than the default one. I added a unit test to simulate what the spring boot one does.

@masseyke masseyke added the v8.2.0 label Feb 3, 2022
Copy link
Member

@jbaiera jbaiera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@masseyke masseyke merged commit 77911f4 into elastic:master Feb 7, 2022
@masseyke masseyke deleted the fix/spring-boot-loading branch February 7, 2022 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Version check fails on SpringBoot applications
2 participants