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:fix typo of interval #3717
Conversation
* you may use this to override global config of "client.rm.lock.retryInterval" | ||
* note: 0 or negative number will take no effect(which mean fall back to global config) | ||
* @return lock retry internal | ||
*/ | ||
int lockRetryInternal() default 0; |
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.
1.5.0先不删除,先做个@deprecated ?
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.
再加个@AliasFor ?
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.
OK
全局搜索了一下, 好像还有很多个地方要同步改一下的 |
Codecov Report
@@ Coverage Diff @@
## develop #3717 +/- ##
=============================================
- Coverage 51.88% 51.88% -0.01%
Complexity 3533 3533
=============================================
Files 639 639
Lines 21414 21414
Branches 2665 2665
=============================================
- Hits 11111 11110 -1
- Misses 9193 9194 +1
Partials 1110 1110
|
Will correct them all. |
rm-datasource/src/main/java/io/seata/rm/datasource/exec/LockRetryController.java
Show resolved
Hide resolved
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
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
fix typo of interval
Ⅱ. 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