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

bugfix: memory visibility of active attribute in file mode #2466

Merged
merged 2 commits into from
Mar 31, 2020

Conversation

ph3636
Copy link
Contributor

@ph3636 ph3636 commented Mar 27, 2020

Ⅰ. Describe what this PR did

bugfix: memory visibility of active attribute in file mode

close the session, then no more branch can be registered.

Ⅱ. 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-io
Copy link

codecov-io commented Mar 27, 2020

Codecov Report

Merging #2466 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #2466      +/-   ##
=============================================
- Coverage      51.46%   51.45%   -0.01%     
+ Complexity      2659     2657       -2     
=============================================
  Files            523      523              
  Lines          16940    16938       -2     
  Branches        2052     2052              
=============================================
- Hits            8718     8716       -2     
+ Misses          7393     7391       -2     
- Partials         829      831       +2
Impacted Files Coverage Δ Complexity Δ
...in/java/io/seata/server/session/GlobalSession.java 85.32% <100%> (-0.46%) 71 <1> (-1)
...o/seata/server/coordinator/DefaultCoordinator.java 54.08% <0%> (-0.52%) 28% <0%> (-1%)
...ata/core/store/db/AbstractDataSourceGenerator.java 0% <0%> (ø) 0% <0%> (ø) ⬇️

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

@zjinlei zjinlei added the bug label Mar 29, 2020
Copy link
Contributor

@zjinlei zjinlei left a comment

Choose a reason for hiding this comment

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

LGTM

@zjinlei
Copy link
Contributor

zjinlei commented Mar 31, 2020

retry CI

@zjinlei zjinlei closed this Mar 31, 2020
@zjinlei zjinlei reopened this Mar 31, 2020
@zjinlei zjinlei merged commit 6c46de7 into apache:develop Mar 31, 2020
@slievrly slievrly added this to the 1.2.0 milestone Apr 1, 2020
@ph3636 ph3636 deleted the active_volatile_file branch April 6, 2020 13:01
@wangliang181230 wangliang181230 added type: bug Category issues or prs related to bug. and removed bug labels Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Category issues or prs related to bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants