-
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: remove H2 and pgsql get primary index code and close resultSet #1535
Conversation
Signed-off-by: slievrly <slievrly@163.com>
Signed-off-by: slievrly <slievrly@163.com>
Codecov Report
@@ Coverage Diff @@
## develop #1535 +/- ##
=============================================
+ Coverage 46.41% 46.46% +0.04%
- Complexity 1692 1693 +1
=============================================
Files 347 347
Lines 12719 12711 -8
Branches 1591 1586 -5
=============================================
+ Hits 5904 5906 +2
+ Misses 6173 6166 -7
+ Partials 642 639 -3
Continue to review full report at Codecov.
|
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
@@ -113,99 +112,97 @@ private static TableMeta fetchSchemeInDefaultWay(DataSource dataSource, String t | |||
|
|||
private static TableMeta resultSetMetaToSchema(ResultSetMetaData rsmd, DatabaseMetaData dbmd, String tableName) |
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.
rsmd 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.
fixed
@@ -89,8 +89,7 @@ private static TableMeta fetchSchema(DataSource dataSource, String tableName) th | |||
return fetchSchemeInDefaultWay(dataSource, tableName); | |||
} | |||
|
|||
private static TableMeta fetchSchemeInDefaultWay(DataSource dataSource, String tableName) | |||
throws SQLException { | |||
private static TableMeta fetchSchemeInDefaultWay(DataSource dataSource, String tableName) throws SQLException { | |||
Connection conn = null; |
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.
close conn
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.
great found.
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.
Signed-off-by: slievrly <slievrly@163.com>
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
optimize: remove H2 and pgsql get primary index code and close resultSet
Ⅱ. Does this pull request fix one issue?
fix #1534
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews