dubbo icon indicating copy to clipboard operation
dubbo copied to clipboard

Enhance serializable check option

Open AlbumenJ opened this issue 2 years ago • 2 comments

What is the purpose of the change

Brief changelog

Verifying this change

Checklist

  • [x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
  • [ ] Each commit in the pull request should have a meaningful subject line and body.
  • [ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • [ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
  • [ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
  • [ ] Add some description to dubbo-website project if you are requesting to add a feature.
  • [ ] GitHub Actions works fine on your own branch.
  • [ ] If this contribution is large, please follow the Software Donation Guide.

AlbumenJ avatar Feb 05 '23 01:02 AlbumenJ

Codecov Report

Merging #11460 (36e2247) into 3.1 (34b92b0) will decrease coverage by 0.05%. The diff coverage is 6.45%.

@@             Coverage Diff              @@
##                3.1   #11460      +/-   ##
============================================
- Coverage     64.67%   64.62%   -0.05%     
  Complexity       14       14              
============================================
  Files          1431     1431              
  Lines         59969    59995      +26     
  Branches       8807     8815       +8     
============================================
- Hits          38786    38774      -12     
- Misses        17111    17142      +31     
- Partials       4072     4079       +7     
Impacted Files Coverage Δ
...bbo/common/utils/DefaultSerializeClassChecker.java 90.80% <0.00%> (-1.06%) :arrow_down:
...mmon/serialize/fastjson2/FastJson2ObjectInput.java 0.00% <0.00%> (ø)
...mon/serialize/fastjson2/FastJson2ObjectOutput.java 0.00% <0.00%> (ø)
...on/serialize/fastjson2/FastJson2Serialization.java 9.52% <0.00%> (-2.25%) :arrow_down:
.../serialize/fastjson2/Fastjson2SecurityManager.java 43.54% <ø> (ø)
.../serialize/hessian2/Hessian2SerializerFactory.java 37.50% <16.66%> (-62.50%) :arrow_down:
...g/apache/dubbo/remoting/api/ConnectionHandler.java 51.51% <0.00%> (-9.10%) :arrow_down:
...e/dubbo/remoting/transport/netty/NettyChannel.java 51.13% <0.00%> (-7.96%) :arrow_down:
.../apache/dubbo/remoting/transport/AbstractPeer.java 67.39% <0.00%> (-4.35%) :arrow_down:
...pache/dubbo/remoting/transport/AbstractServer.java 61.03% <0.00%> (-3.90%) :arrow_down:
... and 8 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov-commenter avatar Feb 05 '23 02:02 codecov-commenter