-
Notifications
You must be signed in to change notification settings - Fork 26.4k
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
[Triple] Fix boxed type methods #11577
Merged
Merged
Conversation
This file contains 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
Partially overlaps with pr #11572 |
Kudos, SonarCloud Quality Gate passed! |
Codecov Report
@@ Coverage Diff @@
## 3.1 #11577 +/- ##
============================================
+ Coverage 64.66% 64.69% +0.03%
Complexity 14 14
============================================
Files 1431 1430 -1
Lines 60014 60016 +2
Branches 8815 8822 +7
============================================
+ Hits 38807 38829 +22
+ Misses 17131 17114 -17
+ Partials 4076 4073 -3
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
EarthChen
approved these changes
Feb 16, 2023
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.
LGTM
mytang0
added a commit
to mytang0/dubbo
that referenced
this pull request
Feb 20, 2023
* origin/3.2: Update codecov config (apache#11582) [Triple] Fix boxed type methods (apache#11577)
mytang0
added a commit
to mytang0/dubbo
that referenced
this pull request
Feb 20, 2023
* origin/3.2: (22 commits) Update codecov config (apache#11582) metric package structure optimization (apache#11576) Fix compile Provides the public part of metrics data collection and export (apache#11522) [Triple] Fix boxed type methods (apache#11577) Bump to 3.2.0-beta.6-SNAPSHOT Fix synchronized in ModelEnvironment (apache#11584) Fix conflict Fix concurrent issue in scope model (apache#11525) Update codecov config (apache#11580) Set timeout value to string (apache#11565) Skip mapping retry if metadata config is invalid (apache#11323) Fix stackoverflow in SerializeSecurityConfigurator (apache#11561) Revert clear response operation of timeoutfilter (apache#11562) Fix hessian2 serializable check (apache#11573) feat: fix oom (apache#11571) Enhance serializable check option (apache#11460) Fix config absent when refresh (apache#11505) Enhance json util check (apache#11501) classNotFound (apache#11552) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What is the purpose of the change
Fix triple protocol's wrapper mode can not recognize primitive method.
When client wants to invoke
call0
with argument1
,Server will call
call1
with(Integer)1
Brief changelog
MultipleSerialization
interface.Verifying this change
Checklist