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

Thin jar: publish assembly jar (fat-jar) with classifier 'assembly' #741

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zzeekk
Copy link

@zzeekk zzeekk commented May 25, 2023

see #696, #697

Like that the main artifact is a thin-jar, usable as maven library dependency, having it's dependencies described in the maven pom.
…e test. Remove slf4j dependency as it is not used.
@zzeekk zzeekk changed the title 696 thin jar Thin jar: publish assembly jar (fat-jar) with classifier 'assembly' May 25, 2023
@zzeekk
Copy link
Author

zzeekk commented May 25, 2023

Hi @nightscape, it would be nice if you could change to publish the default artifact as thin jar with an additional fat-jar artifact with classifier "assembly". WDYT?
Shall i try to fix the conflicts and make a PR into this branch, or do you want to have a look on it?

@nightscape
Copy link
Collaborator

@zzeekk it would be great if you could rebase the PR and update the documentation. Please also try if the new instructions work, both in the command-line via spark-shell, as well as in a SBT/Mill/Gradle build (one should suffice).
Thank you! 👍👍

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.

3 participants