-
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
Fix protobuf class deep copy issue #13585
Closed
namelessssssssssss
wants to merge
20
commits into
apache:3.2
from
namelessssssssssss:3.2-fix-triple-injvm
Closed
Fix protobuf class deep copy issue #13585
namelessssssssssss
wants to merge
20
commits into
apache:3.2
from
namelessssssssssss:3.2-fix-triple-injvm
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
Contributor
namelessssssssssss
commented
Dec 26, 2023
•
edited
Loading
edited
- fix Got exception when running triple protocol with injvm mode, dubbo 3.2.9, jdk 17. #13497
- fix Class cast exception running triple with injvm mode, dubbo 3.3.0-beta.1, jdk 17 #13498
- fix Triple Dubbo dual protocol parameters are missing #13158
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 3.2 #13585 +/- ##
==========================================
- Coverage 70.39% 70.32% -0.08%
==========================================
Files 1606 1609 +3
Lines 69987 70048 +61
Branches 10097 10097
==========================================
- Hits 49266 49258 -8
- Misses 16089 16158 +69
Partials 4632 4632 ☔ View full report in Codecov by Sentry. |
AlbumenJ
requested changes
Dec 27, 2023
dubbo-rpc/dubbo-rpc-injvm/src/main/java/org/apache/dubbo/rpc/protocol/injvm/InjvmInvoker.java
Outdated
Show resolved
Hide resolved
Quality Gate passedThe SonarCloud Quality Gate passed, but some issues were introduced. 5 New issues |
namelessssssssssss
changed the title
Provides new ParamDeepCopyUtil to support protobuf class
Fix protobuf class deep copy issue
Jan 2, 2024
This pull request was closed.
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.