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: add io.seata package shade #6302

Merged
merged 22 commits into from
Mar 2, 2024
Merged

Conversation

slievrly
Copy link
Member

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

add io.seata package shade

Ⅱ. 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

@codecov-commenter
Copy link

codecov-commenter commented Jan 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.02%. Comparing base (ec3a496) to head (f750498).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #6302      +/-   ##
============================================
- Coverage     51.05%   51.02%   -0.04%     
  Complexity     5263     5263              
============================================
  Files           934      934              
  Lines         33026    33026              
  Branches       4002     4002              
============================================
- Hits          16862    16852      -10     
- Misses        14496    14508      +12     
+ Partials       1668     1666       -2     

see 2 files with indirect coverage changes

@@ -30,7 +30,7 @@

<dependencies>
<dependency>
<groupId>${project.groupId}</groupId>
<groupId>org.apache.seata</groupId>
Copy link
Contributor

Choose a reason for hiding this comment

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

如果他不依赖org.apache.seata:seata-all,会影响应用使用吗?假设我使用一个io.seata:seata-all 2.1.0-SNAPSHOT
If he doesn't depend on org.apache.seata:seata-all, will it affect the application usage? Let's say I use an io.seata:seata-all 2.1.0-SNAPSHOT

Copy link
Contributor

@funky-eyes funky-eyes Jan 25, 2024

Choose a reason for hiding this comment

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

In my judgment, if the compatible module doesn't depend on org.apache.seata:seata-all, then it's not complete and users will have problems using it because of the missing classes. If he's missing classes, what's the difference between that and just using io.seata:seata-all and typing a version specifically to be used as a usedby tag?
根据我的判断,如果兼容模块不依赖于org. apache.seata:seata-all,那么它就不完整,用户会因为缺少类而在使用它时遇到问题。如果他缺少类,这与仅使用io.seata:seata-all并输入专门用作usedby标记的版本有什么区别?

Copy link
Member Author

Choose a reason for hiding this comment

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

io.seata:seata-all won't release module package.

@funky-eyes
Copy link
Contributor

image

compatible/pom.xml Outdated Show resolved Hide resolved
@slievrly slievrly added this to the 2.1.0 milestone Mar 2, 2024
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

@funky-eyes funky-eyes merged commit e83d49b into apache:2.x Mar 2, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants