sofastack / sofa-rpc

SOFARPC is a high-performance, high-extensibility, production-level Java RPC framework.
https://www.sofastack.tech/sofa-rpc/docs/Home
Apache License 2.0
3.81k stars 1.17k forks source link

support change grpc maxInboundMessageSize #1333

Closed EvenLjj closed 1 year ago

EvenLjj commented 1 year ago

support change grpc maxInboundMessageSize

codecov[bot] commented 1 year ago

Codecov Report

Merging #1333 (8c4e696) into master (bf8a17d) will increase coverage by 0.04%. The diff coverage is 100.00%.

:exclamation: Current head 8c4e696 differs from pull request most recent head 2a764b5. Consider uploading reports for the commit 2a764b5 to get more accurate results

@@             Coverage Diff              @@
##             master    #1333      +/-   ##
============================================
+ Coverage     71.98%   72.02%   +0.04%     
  Complexity      784      784              
============================================
  Files           416      416              
  Lines         17659    17660       +1     
  Branches       2752     2752              
============================================
+ Hits          12711    12720       +9     
+ Misses         3547     3539       -8     
  Partials       1401     1401              
Impacted Files Coverage Δ
...in/java/com/alipay/sofa/rpc/common/RpcOptions.java 100.00% <ø> (ø)
...fa/rpc/transport/triple/TripleClientTransport.java 71.07% <100.00%> (+0.24%) :arrow_up:

... and 4 files with indirect coverage changes

OrezzerO commented 1 year ago
  1. Test is needed to check the limited size.
  2. Do we need to set outbound size ? @EvenLjj
EvenLjj commented 1 year ago
  1. Test is needed to check the limited size.
  2. Do we need to set outbound size ? @EvenLjj

Fixed and supported. @OrezzerO