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: Slimming down for the seata-all.jar #4353

Merged
merged 25 commits into from
Feb 14, 2022

Conversation

wangliang181230
Copy link
Contributor

@wangliang181230 wangliang181230 commented Jan 28, 2022

optimize: Slimming down for the seata-all.jar.

  1. seata-all瘦身:移除 seata-sqlparser-antlrseata-serializer-protobuf,因为他们体积太大了。需要的用户自行引用依赖。另外,优化了用户未引用依赖时的错误提示信息,提示用户引用相应的依赖。
  2. seata-allseata-bom中补充缺失的模块:seata-hsfseata-compressor-zstd

# Conflicts:
#	rm-datasource/src/main/java/io/seata/rm/datasource/util/JdbcUtils.java
@codecov-commenter
Copy link

codecov-commenter commented Jan 28, 2022

Codecov Report

Merging #4353 (39ee42d) into develop (b62f03e) will increase coverage by 0.00%.
The diff coverage is 18.18%.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop    #4353   +/-   ##
==========================================
  Coverage      48.87%   48.87%           
- Complexity      3935     3938    +3     
==========================================
  Files            732      733    +1     
  Lines          24894    24900    +6     
  Branches        3068     3068           
==========================================
+ Hits           12166    12170    +4     
- Misses         11446    11451    +5     
+ Partials        1282     1279    -3     
Impacted Files Coverage Δ
...c/main/java/io/seata/common/ConfigurationKeys.java 0.00% <0.00%> (ø)
.../io/seata/core/rpc/netty/v1/ProtocolV1Decoder.java 0.00% <0.00%> (ø)
.../io/seata/core/rpc/netty/v1/ProtocolV1Encoder.java 0.00% <0.00%> (ø)
...seata/core/serializer/SerializerServiceLoader.java 0.00% <0.00%> (ø)
.../java/io/seata/core/serializer/SerializerType.java 0.00% <ø> (ø)
.../io/seata/rm/datasource/sql/SQLVisitorFactory.java 80.00% <100.00%> (ø)
...in/java/io/seata/server/session/GlobalSession.java 79.31% <0.00%> (+0.43%) ⬆️
...o/seata/server/coordinator/DefaultCoordinator.java 49.78% <0.00%> (+0.43%) ⬆️
...torage/file/store/FileTransactionStoreManager.java 57.41% <0.00%> (+0.64%) ⬆️

@slievrly slievrly added the maven label Feb 10, 2022
Copy link
Member

@slievrly slievrly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@funky-eyes funky-eyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wangliang181230 wangliang181230 merged commit f1be649 into apache:develop Feb 14, 2022
@wangliang181230 wangliang181230 deleted the optimize/seata-all branch February 14, 2022 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants