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: optimize the versions of the dependencies #4227

Merged
merged 109 commits into from
Feb 16, 2022

Conversation

wangliang181230
Copy link
Contributor

@wangliang181230 wangliang181230 commented Dec 23, 2021

optimize: Unified management versions of the dependencies, and upgrade spring-boot
优化:统一管理依赖的版本,并且升级spring-boot到2.4.13

统一管理依赖的版本:

  1. 依赖的版本号,全部放到 seata-dependencies 中。

升级spring-boot:

  1. 统一 springboot 版本到2.4.13,全局引入spring-boot-dependencies,同时在seata-dependencies中移除在spring-boot-dependencies中已存在的依赖。
  2. @Configuration -> @Configuration(proxyBeanMethods = false):spring-boot项目启动时,不对这些配置类进行代理,加快项目启动速度。

说明:全局引入spring-boot的目的,是切换spring-boot版本,可以同时切换由spring-boot-dependencies 引进来的依赖的版本。
可通过切换spring-boot版本,对多个依赖的版本进行兼容性测试。

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.

@wangliang181230
Copy link
Contributor Author

http://seata.io -> https://seata.io

done,剩余的也全部改成https了

@codecov-commenter
Copy link

codecov-commenter commented Dec 23, 2021

Codecov Report

Merging #4227 (1ddc905) into develop (ca8ff3f) will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #4227      +/-   ##
=============================================
+ Coverage      48.93%   49.02%   +0.08%     
  Complexity      3953     3953              
=============================================
  Files            733      733              
  Lines          24968    24968              
  Branches        3087     3088       +1     
=============================================
+ Hits           12219    12240      +21     
+ Misses         11454    11433      -21     
  Partials        1295     1295              
Impacted Files Coverage Δ
...boot/autoconfigure/SeataCoreAutoConfiguration.java 0.00% <ø> (ø)
...ring/boot/autoconfigure/HttpAutoConfiguration.java 0.00% <ø> (ø)
...boot/autoconfigure/SeataSagaAutoConfiguration.java 0.00% <ø> (ø)
...o/seata/server/coordinator/DefaultCoordinator.java 52.55% <0.00%> (+0.36%) ⬆️
...ata/rm/datasource/undo/AbstractUndoLogManager.java 50.27% <0.00%> (+0.54%) ⬆️
...very/registry/zk/ZookeeperRegisterServiceImpl.java 61.76% <0.00%> (+0.73%) ⬆️
.../io/seata/core/rpc/netty/NettyClientBootstrap.java 43.66% <0.00%> (+1.40%) ⬆️
...eata/core/rpc/netty/NettyClientChannelManager.java 71.53% <0.00%> (+1.53%) ⬆️
...ta/spring/annotation/GlobalTransactionScanner.java 30.80% <0.00%> (+2.02%) ⬆️
...seata/rm/datasource/undo/AbstractUndoExecutor.java 70.00% <0.00%> (+2.85%) ⬆️
... and 2 more

@slievrly slievrly added the maven label Feb 10, 2022
@@ -517,16 +452,108 @@
</exclusion>
</exclusions>
</dependency>
<dependency>
Copy link
Member

Choose a reason for hiding this comment

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

It is recommended to declare versions of important dependencies. Spring-boot-dependencies are just an import POM. We should declare versions of important dependencies, such as Redis, Mariadb, These supported versions should not be limited by the scope of Spring-boot-dependencies, especially since Seata-Server also depend on seata-dependencies.

建议声明重要依赖的版本。spring-boot-dependencies 只是个import pom,我们应该声明对重要依赖支持的版本,比如pom中的redis,mariadb,这些支持的版本不应该受限于spring-boot-dependencies 的scope,特别是seata-server也会依赖seata-dependencies。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, the versions of core dependencies is set in server/pom.xml

Copy link
Member

Choose a reason for hiding this comment

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

It is recommended to keep dependencies like mysql, Postgres, H2, mariadb directly in dependencies. pom. The dependency declaration in Dependencies is only dependencyManagement. Have we fully tested the above dependencies in Spring-boot? In fact, our application module does not directly rely on DB-driver dependencies such as mysql, Postgres, H2, Mariadb, etc. except server, but it can indicate which versions we recommend and adapt. Users may be able to manage dependencies using import seta-dependency.

@wangliang181230 wangliang181230 requested review from slievrly and removed request for jsbxyyx February 16, 2022 10:24
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

@slievrly slievrly changed the title optimize: Unified management versions of the dependencies, and upgrade spring-boot optimize: optimize the versions of the dependencies Feb 16, 2022
@slievrly slievrly merged commit 314ec7c into apache:develop Feb 16, 2022
@wangliang181230 wangliang181230 deleted the optimize/build branch February 16, 2022 13:55
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.

3 participants