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: abstract table meta cache #1750

Merged
merged 10 commits into from Oct 14, 2019

Conversation

github-ygy
Copy link
Contributor

Ⅰ. Describe what this PR did

abstract table meta cache

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

Codecov Report

Merging #1750 into develop will increase coverage by 0.21%.
The diff coverage is 83.09%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #1750      +/-   ##
=============================================
+ Coverage      50.36%   50.57%   +0.21%     
- Complexity      2159     2166       +7     
=============================================
  Files            412      414       +2     
  Lines          14219    14192      -27     
  Branches        1714     1707       -7     
=============================================
+ Hits            7161     7178      +17     
+ Misses          6367     6324      -43     
+ Partials         691      690       -1
Impacted Files Coverage Δ Complexity Δ
.../rm/datasource/exec/BaseTransactionalExecutor.java 18.96% <0%> (ø) 8 <0> (ø) ⬇️
...ata/rm/datasource/undo/AbstractUndoLogManager.java 23.49% <0%> (+0.55%) 10 <0> (ø) ⬇️
...n/java/io/seata/rm/datasource/DataSourceProxy.java 68.75% <0%> (+2.08%) 9 <0> (-1) ⬇️
...o/seata/rm/datasource/AbstractConnectionProxy.java 12.08% <0%> (+0.38%) 6 <0> (ø) ⬇️
...asource/sql/struct/cache/OracleTableMetaCache.java 6.38% <83.33%> (ø) 3 <3> (?)
...m/datasource/sql/struct/TableMetaCacheFactory.java 83.33% <83.33%> (ø) 3 <3> (?)
...tasource/sql/struct/cache/MysqlTableMetaCache.java 86.66% <86.66%> (ø) 10 <10> (?)
...ource/sql/struct/cache/AbstractTableMetaCache.java 88.23% <88.23%> (ø) 10 <10> (?)
... and 2 more

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 0fe51a0...35e3ca3. Read the comment docs.

xingfudeshi and others added 3 commits October 11, 2019 17:44
…_abstract_table_cache

# Conflicts:
#	rm-datasource/src/main/java/io/seata/rm/datasource/undo/AbstractUndoLogManager.java
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.

whether need add oracle meta cache test case

@github-ygy
Copy link
Contributor Author

github-ygy commented Oct 14, 2019

whether need add oracle meta cache test case

whether need add oracle meta cache test case

I don't know the oracle yet. I found it needs to be sorted out from the bug of pr1749. I'll learn later that can provide Oracle tests, but now I can provide TableMetaCacheFactoryTest.

@jsbxyyx
Copy link
Member

jsbxyyx commented Oct 14, 2019

whether need add oracle meta cache test case

whether need add oracle meta cache test case

I don't know the oracle yet. I found it needs to be sorted out from the bug of pr1749. I'll learn later that can provide Oracle tests, but now I can provide TableMetaCacheFactoryTest.

Sorry, I am not expressing it right.

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

@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

@jsbxyyx jsbxyyx merged commit f405c5e into apache:develop Oct 14, 2019
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)
@wangliang181230 wangliang181230 added this to the 0.9.0 milestone Aug 9, 2021
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

6 participants