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

[improve][misc] Pin Netty version in pulsar-io/alluxio #21728

Merged
merged 1 commit into from Dec 15, 2023

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Dec 14, 2023

Motivation

Upgrade to Netty 4.1.103.Final fails with errors in pulsar-io/alluxio (see comments in #21723). Prepare for the upgrade by pinning the Netty version in pulsar-io/alluxio.

Modifications

  • Use dependencyManagement to pin versions
  • Use 4.1.100.Final since it should be compatible

Documentation

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

- Use dependencyManagement to pin versions
- Use 4.1.100.Final since it should be compatible
- Prepare for upgrade to 4.1.103.Final
@lhotari lhotari added area/dependency Pull requests that update a dependency file ready-to-test labels Dec 14, 2023
@lhotari lhotari self-assigned this Dec 14, 2023
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Dec 14, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #21728 (a625f6a) into master (58ae086) will increase coverage by 36.69%.
Report is 2 commits behind head on master.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #21728       +/-   ##
=============================================
+ Coverage     36.67%   73.37%   +36.69%     
- Complexity    12255    32753    +20498     
=============================================
  Files          1717     1893      +176     
  Lines        131212   140761     +9549     
  Branches      14342    15509     +1167     
=============================================
+ Hits          48123   103284    +55161     
+ Misses        76691    29366    -47325     
- Partials       6398     8111     +1713     
Flag Coverage Δ
inttests 24.13% <40.00%> (-0.03%) ⬇️
systests 24.69% <40.00%> (+0.02%) ⬆️
unittests 72.64% <100.00%> (+40.84%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...sar/broker/service/persistent/PersistentTopic.java 78.63% <100.00%> (+26.59%) ⬆️

... and 1462 files with indirect coverage changes

@merlimat merlimat merged commit b777136 into apache:master Dec 15, 2023
49 of 50 checks passed
@merlimat merlimat added this to the 3.2.0 milestone Dec 15, 2023
lhotari added a commit that referenced this pull request Mar 28, 2024
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 15, 2024
(cherry picked from commit b777136)
(cherry picked from commit d3f0a4b)
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 17, 2024
(cherry picked from commit b777136)
(cherry picked from commit d3f0a4b)
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 17, 2024
(cherry picked from commit b777136)
(cherry picked from commit d3f0a4b)
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 19, 2024
(cherry picked from commit b777136)
(cherry picked from commit d3f0a4b)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 23, 2024
(cherry picked from commit b777136)
(cherry picked from commit d3f0a4b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependency Pull requests that update a dependency file cherry-picked/branch-3.0 doc-not-needed Your PR changes do not impact docs ready-to-test release/3.0.4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants