Skip to content

KAFKA-17052: Downgrade ShadowJarPlugin to 8.1.3#16489

Merged
omkreddy merged 1 commit intoapache:trunkfrom
gharris1727:kafka-17052
Jun 29, 2024
Merged

KAFKA-17052: Downgrade ShadowJarPlugin to 8.1.3#16489
omkreddy merged 1 commit intoapache:trunkfrom
gharris1727:kafka-17052

Conversation

@gharris1727
Copy link
Contributor

This version of the ShadowJarPlugin uses the incorrect classifier for the published archive. This is a temporary measure to fix publishing prior to the upcoming release.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

Copy link
Contributor

@omkreddy omkreddy left a comment

Choose a reason for hiding this comment

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

@gharris1727 Thanks for the PR. LGTM

@omkreddy omkreddy merged commit a33cc3d into apache:trunk Jun 29, 2024
abhi-ksolves pushed a commit to ksolves/kafka that referenced this pull request Jul 31, 2024
This version of the ShadowJarPlugin uses the incorrect classifier for the published archive. This is a temporary measure to fix publishing prior to the upcoming release.

Signed-off-by: Greg Harris <[email protected]>


Reviewers:  Manikumar Reddy <[email protected]>
mumrah pushed a commit that referenced this pull request Mar 12, 2025
**Prologue (related PR's):**
-  #16295
-  #17218
-  #16489 

**Action points:**
- switch shadow plugin from `io.github.goooler.shadow` to
`com.gradleup.shadow`
- upgrade plugin version from **8.1.3** to **8.3.5** (release notes:
https://gradleup.com/shadow/changes/#v8-3-5-2024-11-03)

**Rationale:** both previous one and currently used shadow plugins are
now deprecated (in favor of `com.gradleup.shadow`):
- `com.github.johnrengelman.shadow` maintenance was transferred to
`com.gradleup.shadow`:
https://github.com/GradleUp/shadow/tree/8.3.5?tab=readme-ov-file#gradle-shadow
- `io.github.goooler.shadow`: changes are ported to
`com.gradleup.shadow`:
https://github.com/Goooler/shadow?tab=readme-ov-file#gradle-shadow

**Pitfall (to keep in mind):** 
- #15532 

Co-authored-by: Goooler <[email protected]>
Reviewers: Apoorv Mittal <[email protected]>, David Arthur <[email protected]>
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.

2 participants