[improve][misc] Update caffeine from 2.9.1 to 3.1.2 #18647
Merged
Conversation
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
|
@shoothzj Please add the following content to your PR description and select a checkbox: |
codelipenghui
approved these changes
Nov 28, 2022
codelipenghui
added
doc-not-needed
Contributor
|
/pulsarbot run-failure-checks |
github-actions
bot
added
doc-label-missing
doc-not-needed
hezhangjian
changed the title
hezhangjian
changed the title
hezhangjian
changed the title
Codecov Report
@@ Coverage Diff @@
## master #18647 +/- ##
============================================
- Coverage 47.51% 47.30% -0.22%
+ Complexity 10520 10461 -59
============================================
Files 698 698
Lines 68079 68079
Branches 7280 7280
============================================
- Hits 32351 32203 -148
- Misses 32151 32303 +152
+ Partials 3577 3573 -4
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Technoboy-
changed the title
hezhangjian
changed the title
Technoboy-
approved these changes
Nov 28, 2022
Jason918
approved these changes
Nov 28, 2022
BewareMyPower
approved these changes
Nov 28, 2022
codelipenghui
added a commit
to codelipenghui/incubator-pulsar
that referenced
this pull request
Dec 6, 2022
2 tasks
codelipenghui
added a commit
that referenced
this pull request
Dec 6, 2022
lifepuzzlefun
pushed a commit
to lifepuzzlefun/pulsar
that referenced
this pull request
Dec 9, 2022
### Motivation Now we only use caffeine in broker module. *caffeine* has a jdk11+ version, which is more proper for jdk11+ project to integrated with.(Spring3.0 uses caffeine 3.X too.) So I think it's worth for us to update caffeine to 3.x ### Modifications Update caffeine from 2.9.1 to 3.1.2 - [ ] Anything that affects deployment ### Documentation <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. --> - [ ] `doc` <!-- Your PR contains doc changes. Please attach the local preview screenshots (run `sh start.sh` at `pulsar/site2/website`) to your PR description, or else your PR might not get merged. --> - [ ] `doc-required` <!-- Your PR changes impact docs and you will update later --> - [x] `doc-not-needed` <!-- Your PR changes do not impact docs --> - [ ] `doc-complete` <!-- Docs have been already added --> ### Matching PR in forked repository apache#18647
lifepuzzlefun
pushed a commit
to lifepuzzlefun/pulsar
that referenced
this pull request
Jan 10, 2023
### Motivation Now we only use caffeine in broker module. *caffeine* has a jdk11+ version, which is more proper for jdk11+ project to integrated with.(Spring3.0 uses caffeine 3.X too.) So I think it's worth for us to update caffeine to 3.x ### Modifications Update caffeine from 2.9.1 to 3.1.2 - [ ] Anything that affects deployment ### Documentation <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. --> - [ ] `doc` <!-- Your PR contains doc changes. Please attach the local preview screenshots (run `sh start.sh` at `pulsar/site2/website`) to your PR description, or else your PR might not get merged. --> - [ ] `doc-required` <!-- Your PR changes impact docs and you will update later --> - [x] `doc-not-needed` <!-- Your PR changes do not impact docs --> - [ ] `doc-complete` <!-- Docs have been already added --> ### Matching PR in forked repository apache#18647
lifepuzzlefun
pushed a commit
to lifepuzzlefun/pulsar
that referenced
this pull request
Jan 10, 2023
lhotari
pushed a commit
to apache/pulsar-sql
that referenced
this pull request
Oct 18, 2024
### Motivation Now we only use caffeine in broker module. *caffeine* has a jdk11+ version, which is more proper for jdk11+ project to integrated with.(Spring3.0 uses caffeine 3.X too.) So I think it's worth for us to update caffeine to 3.x ### Modifications Update caffeine from 2.9.1 to 3.1.2 - [ ] Anything that affects deployment ### Documentation <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. --> - [ ] `doc` <!-- Your PR contains doc changes. Please attach the local preview screenshots (run `sh start.sh` at `pulsar/site2/website`) to your PR description, or else your PR might not get merged. --> - [ ] `doc-required` <!-- Your PR changes impact docs and you will update later --> - [x] `doc-not-needed` <!-- Your PR changes do not impact docs --> - [ ] `doc-complete` <!-- Docs have been already added --> ### Matching PR in forked repository apache/pulsar#18647
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
Motivation
Now we only use caffeine in broker module. caffeine has a jdk11+ version, which is more proper for jdk11+ project to integrated with.(Spring3.0 uses caffeine 3.X too.) So I think it's worth for us to update caffeine to 3.x
Modifications
Update caffeine from 2.9.1 to 3.1.2
Documentation
docdoc-requireddoc-not-neededdoc-completeMatching PR in forked repository
#18647