Skip to content

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Oct 2, 2025

Motivation

Apache Parent POM was updated from version 29 to 35 in PR #24742. It upgraded maven-source-plugin from version 3.2.1 to 3.3.1. That broke mvn deploy with this type of error message:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-source-plugin:3.3.1:jar-no-fork (attach-sources) on project pulsar-client-all: Presumably you have configured maven-source-plugin to execute twice in your build. You have to configure a classifier for at least one of them. -> [Help 1]

Modifications

Set attach-sources phase to none for maven-source-plugin when maven-shade-plugin has been configured with <createSourcesJar>true</createSourcesJar>. This currently applies to pulsar-client-all module.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@lhotari lhotari added this to the 4.2.0 milestone Oct 2, 2025
@lhotari lhotari self-assigned this Oct 2, 2025
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Oct 2, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.27%. Comparing base (771ce41) to head (e587395).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #24811      +/-   ##
============================================
- Coverage     74.44%   74.27%   -0.18%     
+ Complexity    33859    33422     -437     
============================================
  Files          1912     1912              
  Lines        149072   149072              
  Branches      17299    17299              
============================================
- Hits         110981   110719     -262     
- Misses        29292    29520     +228     
- Partials       8799     8833      +34     
Flag Coverage Δ
inttests 26.31% <ø> (-0.41%) ⬇️
systests 22.73% <ø> (-0.12%) ⬇️
unittests 73.79% <ø> (-0.19%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 87 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lhotari lhotari merged commit f5547dc into apache:master Oct 2, 2025
98 of 102 checks passed
LucasEby pushed a commit to LucasEby/pulsar that referenced this pull request Oct 3, 2025
lhotari added a commit that referenced this pull request Oct 4, 2025
lhotari added a commit that referenced this pull request Oct 4, 2025
lhotari added a commit that referenced this pull request Oct 4, 2025
walkinggo pushed a commit to walkinggo/pulsar that referenced this pull request Oct 8, 2025
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request Oct 15, 2025
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Nov 6, 2025
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.

3 participants