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

refactor: 优化对sqlserver的支持 #63

Merged
merged 1 commit into from
Dec 20, 2023
Merged

Conversation

zhou-hao
Copy link
Member

  1. 优化分页支持
  2. 优化表结构解析
  3. 增加批量upsert支持
  4. 增加索引支持

1. 优化分页支持
2. 优化表结构解析
3. 增加批量upsert支持
4. 增加索引支持
@codecov-commenter
Copy link

Codecov Report

Attention: 59 lines in your changes are missing coverage. Please review.

Comparison is base (0a2c7e6) 61.97% compared to head (64c2707) 62.21%.

Files Patch % Lines
...b/supports/mssql/SqlServerBatchUpsertOperator.java 72.85% 23 Missing and 15 partials ⚠️
...b/supports/mssql/SqlServerIndexMetadataParser.java 58.13% 16 Missing and 2 partials ⚠️
...pports/mssql/SqlServer2012TableMetadataParser.java 33.33% 2 Missing ⚠️
...orm/rdb/supports/mssql/SqlServer2012Paginator.java 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #63      +/-   ##
============================================
+ Coverage     61.97%   62.21%   +0.23%     
- Complexity     2057     2069      +12     
============================================
  Files           325      327       +2     
  Lines          8392     8580     +188     
  Branches        783      815      +32     
============================================
+ Hits           5201     5338     +137     
- Misses         2779     2812      +33     
- Partials        412      430      +18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zhou-hao zhou-hao merged commit e2805d4 into master Dec 20, 2023
1 check passed
@zhou-hao zhou-hao deleted the refactor-sql-server branch December 20, 2023 06:56
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.

2 participants