Skip to content

Conversation

@heliang666s
Copy link
Member

@heliang666s heliang666s commented Aug 1, 2025

#15597 feat:Implement protobuf zero-copy and Implement zero-copy serialization for POJOs by bypassing Protobuf wrapper

@heliang666s heliang666s requested a review from EarthChen August 1, 2025 10:22
@heliang666s heliang666s added this to the 3.3.6 milestone Aug 1, 2025
@heliang666s heliang666s added the help wanted Everything needs help from contributors label Aug 1, 2025
@codecov-commenter
Copy link

codecov-commenter commented Aug 1, 2025

Codecov Report

❌ Patch coverage is 28.81356% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.06%. Comparing base (af2625c) to head (aa1c72d).
⚠️ Report is 6 commits behind head on 3.3.

⚠️ Current head aa1c72d differs from pull request most recent head b3fd98e

Please upload reports for the commit b3fd98e to get more accurate results.

Files with missing lines Patch % Lines
...bbo/rpc/protocol/tri/ReflectionPackableMethod.java 0.00% 27 Missing ⚠️
...ava/org/apache/dubbo/rpc/model/PackableMethod.java 0.00% 8 Missing ⚠️
...g/apache/dubbo/rpc/protocol/tri/PbArrayPacker.java 0.00% 5 Missing ⚠️
...va/org/apache/dubbo/rpc/protocol/tri/PbUnpack.java 0.00% 1 Missing ⚠️
.../rpc/protocol/tri/h12/grpc/GrpcCompositeCodec.java 91.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                3.3   #15605      +/-   ##
============================================
- Coverage     61.06%   61.06%   -0.01%     
- Complexity    11693    11706      +13     
============================================
  Files          1909     1911       +2     
  Lines         86783    86837      +54     
  Branches      13094    13100       +6     
============================================
+ Hits          52994    53026      +32     
- Misses        28373    28398      +25     
+ Partials       5416     5413       -3     
Flag Coverage Δ
integration-tests-java21 33.05% <28.81%> (+0.06%) ⬆️
integration-tests-java8 33.05% <28.81%> (-0.01%) ⬇️
samples-tests-java21 32.71% <0.00%> (-0.02%) ⬇️
samples-tests-java8 30.31% <0.00%> (-0.14%) ⬇️
unit-tests-java11 59.06% <28.81%> (-0.02%) ⬇️
unit-tests-java17 58.79% <28.81%> (+0.01%) ⬆️
unit-tests-java21 58.77% <28.81%> (+<0.01%) ⬆️
unit-tests-java8 59.06% <28.81%> (+<0.01%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@heliang666s heliang666s requested review from asf-gitbox and oxsean and removed request for asf-gitbox August 1, 2025 10:52
@heliang666s heliang666s changed the title feat:protobuf zero-copy feat:Implement protobuf zero-copy and Implement zero-copy serialization for POJOs by bypassing Protobuf wrapper Aug 2, 2025
@heliang666s heliang666s marked this pull request as draft August 6, 2025 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Everything needs help from contributors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants