-
Notifications
You must be signed in to change notification settings - Fork 333
Reuse shadowJar for spark client bundle jar maven publish #1857
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
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
020334e
fix spark client
gh-yzou 33e3fde
fix test failure and address feedback
gh-yzou dbb23e1
fix error
gh-yzou 8556976
update regression test
gh-yzou 028cc36
update classifier name
gh-yzou 4c8a9fb
address comment
gh-yzou f020db3
add change
gh-yzou 747b6f0
update doc
gh-yzou df33614
update build and readme
gh-yzou c011531
add back jr
gh-yzou 7aa6a26
udpate dependency
gh-yzou 5979e2b
add change
gh-yzou a1f892c
update
gh-yzou cd2a94e
update tests
gh-yzou 1fb7ccd
remove merge service file
gh-yzou 841bcc4
update readme
gh-yzou 5ad378f
update readme
gh-yzou File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit:
:assembleis higher level (less internal details visible to end users) and does not have much overhead compared toshadowJar.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is still good to let user know that there is a specific task to just produce the jar in case they don't want to run assemble or build. I added one sentence below to mention that this task is also executed when run
gradlew assembleorgradlew build. so user can choose whatever way they want.