Skip to content
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

optimize: some minor syntax optimization #5869

Merged
merged 2 commits into from Sep 24, 2023
Merged

Conversation

slievrly
Copy link
Member

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

some minor syntax optimization

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

Signed-off-by: slievrly <slievrly@163.com>
@codecov
Copy link

codecov bot commented Sep 24, 2023

Codecov Report

Merging #5869 (e1703cd) into 2.x (9fe8d3b) will increase coverage by 0.04%.
Report is 1 commits behind head on 2.x.
The diff coverage is 57.89%.

❗ Current head e1703cd differs from pull request most recent head 2f02794. Consider uploading reports for the commit 2f02794 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #5869      +/-   ##
============================================
+ Coverage     48.88%   48.93%   +0.04%     
- Complexity     4461     4469       +8     
============================================
  Files           821      821              
  Lines         28641    28641              
  Branches       3571     3571              
============================================
+ Hits          14002    14015      +13     
+ Misses        13181    13173       -8     
+ Partials       1458     1453       -5     
Files Changed Coverage Δ
...on/tx/api/remoting/parser/DubboRemotingParser.java 0.00% <0.00%> (ø)
...tion/tx/api/remoting/parser/HSFRemotingParser.java 0.00% <0.00%> (ø)
.../tx/api/remoting/parser/SofaRpcRemotingParser.java 0.00% <0.00%> (ø)
...statelang/parser/impl/AbstractTaskStateParser.java 71.42% <0.00%> (ø)
.../saga/statelang/parser/impl/JacksonJsonParser.java 43.75% <0.00%> (ø)
...ava/io/seata/serializer/seata/SeataSerializer.java 80.00% <0.00%> (ø)
.../src/main/java/io/seata/common/util/BlobUtils.java 60.00% <100.00%> (ø)
...main/java/io/seata/common/util/ReflectionUtil.java 72.52% <100.00%> (ø)
...ava/io/seata/compressor/deflater/DeflaterUtil.java 80.64% <100.00%> (ø)
.../seata/rm/datasource/exec/MultiUpdateExecutor.java 80.00% <100.00%> (ø)
... and 4 more

... and 7 files with indirect coverage changes

@slievrly slievrly added this to the 2.0.0 milestone Sep 24, 2023
@slievrly slievrly merged commit 4c6d48e into apache:2.x Sep 24, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant