-
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
bugfix: fix can't refresh table meta data for oracle #1720
bugfix: fix can't refresh table meta data for oracle #1720
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1720 +/- ##
=============================================
+ Coverage 46.99% 48.21% +1.21%
- Complexity 1787 1841 +54
=============================================
Files 365 365
Lines 13298 13276 -22
Branches 1644 1639 -5
=============================================
+ Hits 6250 6401 +151
+ Misses 6369 6215 -154
+ Partials 679 660 -19
Continue to review full report at Codecov.
|
… into can_not_refresh_table_meta_oracle
rm-datasource/src/main/java/io/seata/rm/datasource/DataSourceProxy.java
Outdated
Show resolved
Hide resolved
rm-datasource/src/main/java/io/seata/rm/datasource/sql/struct/TableMetaCacheOracle.java
Outdated
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.
I left some comments.
please remove TableMetaCache#resultSetMetaToSchema(java.sql.ResultSet rs2, AbstractConnectionProxy conn, String tableName) the method is never used. |
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
It is recommended to abstract the public methods of TableMetaCache and TableMetaCacheOracle to a base class BaseTableMetaCache. |
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.
…ble_meta_oracle # Conflicts: # rm-datasource/src/main/java/io/seata/rm/datasource/DataSourceProxy.java
ok |
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 can not refresh table meta data for oracle
Ⅱ. Does this pull request fix one issue?
#1719 #1714
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews