Skip to content
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

Change the POM and CD configs to make agent-core Spark specific #604

Open
wajda opened this issue Feb 16, 2023 · 0 comments
Open

Change the POM and CD configs to make agent-core Spark specific #604

wajda opened this issue Feb 16, 2023 · 0 comments

Comments

@wajda
Copy link
Contributor

wajda commented Feb 16, 2023

Universal Spark version independent agent-core keeps making problems in transient dependency management.
We need to return to the concept of separate core build per major Spark version controlled by Maven profiles.
We could use Maven classifier to distinguish between versions.

https://www.baeldung.com/maven-artifact-classifiers

Reasoning:

  1. No more MethodNotFoundException and other linkage errors because of binary incompatibility in transient dependencies of Spline and Spark
  2. Avoiding dependency shading (e.g. json4s, see AWS Glue 4.0 - NoSuchMethodError #602) that complicates running code in the local development mode and debugging
  3. Clean and unambiguous dependencies and easy profile switching makes the development process easier and less error prone, thus more efficient.
  4. The project structure would allow for solving potential breaking API changes in future Spark versions easier with ability to use additional Spark version specific source directories and adapters if there is a need for.
@wajda wajda added this to the 1.1.0 milestone Feb 16, 2023
@cerveada cerveada modified the milestones: 1.1.0, 1.2.0 Mar 29, 2023
@wajda wajda removed this from the 1.2.0 milestone Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Backlog
Development

No branches or pull requests

2 participants