Skip to content

Conversation

@funky-eyes
Copy link
Contributor

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

fixes #5359

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@funky-eyes funky-eyes added type: bug Category issues or prs related to bug. module/rm-datasource rm-datasource module labels Jan 15, 2024
@funky-eyes funky-eyes added this to the 2.1.0 milestone Jan 15, 2024
@funky-eyes funky-eyes changed the title bugfix: pgsql cluster mode is supported bugfix: AT mode support the URL of a PGSQL cluster Jan 15, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jan 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.99%. Comparing base (82b036f) to head (c5e7b63).
Report is 287 commits behind head on 2.x.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #6261      +/-   ##
============================================
+ Coverage     50.97%   50.99%   +0.01%     
+ Complexity     5091     5088       -3     
============================================
  Files           918      918              
  Lines         32029    32031       +2     
  Branches       3864     3865       +1     
============================================
+ Hits          16328    16334       +6     
+ Misses        14122    14120       -2     
+ Partials       1579     1577       -2     
Files with missing lines Coverage Δ
...n/java/io/seata/rm/datasource/DataSourceProxy.java 71.18% <100.00%> (+10.61%) ⬆️

... and 4 files with indirect coverage changes

@funky-eyes funky-eyes closed this Jan 15, 2024
@funky-eyes funky-eyes reopened this Jan 15, 2024
slievrly
slievrly previously approved these changes Jan 16, 2024
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
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 merged commit a59a39a into apache:2.x Jan 16, 2024
YvCeung pushed a commit to YvCeung/incubator-seata that referenced this pull request Dec 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module/rm-datasource rm-datasource module type: bug Category issues or prs related to bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

seata的at模式,数据源为postgresql的问题

3 participants