-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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: Removed Reloadable from the redis/db SessionManager. #2843
optimize: Removed Reloadable from the redis/db SessionManager. #2843
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2843 +/- ##
=============================================
- Coverage 50.57% 50.57% -0.01%
- Complexity 3105 3107 +2
=============================================
Files 599 599
Lines 19510 19528 +18
Branches 2406 2410 +4
=============================================
+ Hits 9868 9877 +9
- Misses 8650 8657 +7
- Partials 992 994 +2
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
} | ||
break; | ||
case AsyncCommitting: | ||
if (storeMode == StoreMode.FILE) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why only file mode can reload this status of global session?
AsyncCommitting
CommitRetrying
TimeoutRollbackRetrying
Begin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If say so, this hold method just use for file mode?
…-redis-db-session-manager-2
Codecov Report
@@ Coverage Diff @@
## develop #2843 +/- ##
=============================================
- Coverage 51.04% 51.03% -0.01%
- Complexity 3259 3261 +2
=============================================
Files 609 609
Lines 20027 20045 +18
Branches 2501 2504 +3
=============================================
+ Hits 10222 10230 +8
- Misses 8788 8795 +7
- Partials 1017 1020 +3
|
…-redis-db-session-manager-2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…-redis-db-session-manager-2
…r-2' into optimize-redis-db-session-manager-2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ⅰ. Describe what this PR did
从redis和db模式的SessionManager中移除Reloadable接口实现。因为它们并不需要reload。
从PR #2828 拆分出来的。
Ⅱ. 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