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: string splice and release lock #1770

Merged
merged 7 commits into from Oct 16, 2019
Merged

Conversation

zjinlei
Copy link
Contributor

@zjinlei zjinlei commented Oct 15, 2019

Ⅰ. Describe what this PR did

replace + to append
reduce unnecessary method:doReleaseLock

Ⅱ. 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 Oct 15, 2019

Codecov Report

Merging #1770 into develop will increase coverage by <.01%.
The diff coverage is 31.57%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #1770      +/-   ##
=============================================
+ Coverage      50.56%   50.56%   +<.01%     
+ Complexity      2166     2165       -1     
=============================================
  Files            414      414              
  Lines          14196    14195       -1     
  Branches        1709     1709              
=============================================
  Hits            7178     7178              
  Misses          6328     6328              
+ Partials         690      689       -1
Impacted Files Coverage Δ Complexity Δ
...java/io/seata/tm/api/DefaultGlobalTransaction.java 51.61% <ø> (ø) 15 <0> (ø) ⬇️
...ta/rm/datasource/exec/SelectForUpdateExecutor.java 0% <0%> (ø) 0 <0> (ø) ⬇️
.../rm/datasource/undo/mysql/MySQLUndoLogManager.java 10.25% <0%> (ø) 2 <0> (ø) ⬇️
...va/io/seata/rm/datasource/exec/UpdateExecutor.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...va/io/seata/rm/datasource/exec/DeleteExecutor.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...eata/core/rpc/netty/AbstractRpcRemotingClient.java 18.91% <0%> (ø) 6 <0> (ø) ⬇️
...java/io/seata/server/lock/memory/MemoryLocker.java 87.5% <100%> (+0.29%) 21 <5> (+1) ⬆️
...ata/rm/datasource/undo/AbstractUndoLogManager.java 23.49% <100%> (ø) 10 <0> (ø) ⬇️
.../java/io/seata/server/lock/DefaultLockManager.java 44.44% <50%> (-3.48%) 10 <0> (-2)
...server/store/file/FileTransactionStoreManager.java 53.94% <0%> (+0.32%) 27% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29b3248...a770905. Read the comment docs.

Copy link
Contributor

@l81893521 l81893521 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
Copy link
Member

@zjinlei Is there a better description of the pr title?

@zjinlei zjinlei changed the title optimize:fix typo optimize:string splice and release lock Oct 15, 2019
@zjinlei zjinlei changed the title optimize:string splice and release lock optimize: string splice and release lock Oct 15, 2019
Copy link
Member

@jsbxyyx jsbxyyx 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 Do Not Merge Do not merge into develop and removed Do Not Merge Do not merge into develop labels Oct 16, 2019
@zjinlei zjinlei merged commit d9d888d into apache:develop Oct 16, 2019
@zjinlei zjinlei deleted the optimize branch October 16, 2019 09:25
dangquocbang pushed a commit to dangquocbang/seata that referenced this pull request Oct 22, 2019
* 'develop' of https://github.com/seata/seata:
  feature:support gRpc (apache#1492)
  bugfix:fix auto proxy for multiple-datasource does not work (apache#1793)
  optimize: increase rm code coverage by db mock (apache#1674)
  bugfix: mysql can not get primary key value (apache#1788)
  bugfix: jdk 11 remoteAddress is null (apache#1764)
  test: verify that the analysis of the cause of issue 1702 is correct (apache#1710)
  optimize: seata server register eureka instance id (apache#1790)
  optimize: put message to logQueue (apache#1760)
  optimize:optimize rpc remoting log(apache#1787)
  optimize: simplify code (apache#1786)
  version: change version to 1.0.0-SNAPSHOT (apache#1785)
  bugfix: clean up resources in time to avoid mutual influence between unit tests (apache#1778)
  optimize:remove unused method (apache#1766)
  bugfix: DeleteExecutor. buildBeforeImageSQL keyword checker by db type (apache#1777)
  optimize: string splice and release lock (apache#1770)
  release 0.9.0 (apache#1771)
  bugfix: oracle rollback failed when the table has null Blob Clob value (apache#1761)
  bugfix: service method not support interface type parameter (apache#1759)
  optimize: abstract table meta cache (apache#1750)
  optimize: improve the efficiency of the batch log (apache#1743)
@slievrly slievrly added this to the 1.0 milestone Dec 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants