-
Notifications
You must be signed in to change notification settings - Fork 321
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
feat: optimize tidb data integration #3839
feat: optimize tidb data integration #3839
Conversation
SDK Test Report102 files +1 102 suites +1 2m 13s ⏱️ +8s Results for commit d8dff65. ± Comparison against base commit 86d640a. This pull request removes 48 and adds 35 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3839 +/- ##
=============================================
- Coverage 74.87% 40.75% -34.12%
Complexity 658 658
=============================================
Files 742 195 -547
Lines 133925 11679 -122246
Branches 1387 1412 +25
=============================================
- Hits 100277 4760 -95517
+ Misses 33344 6615 -26729
Partials 304 304 ☔ View full report in Codecov by Sentry. |
Linux Test Report 57 files + 4 244 suites +184 1h 43m 59s ⏱️ + 1h 15m 31s Results for commit d8dff65. ± Comparison against base commit 86d640a. ♻️ This comment has been updated with latest results. |
…20100/OpenMLDB into feat/optimize-tidb-data-integration
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
java/openmldb-batch/src/main/scala/com/_4paradigm/openmldb/batch/utils/DataSourceUtil.scala
Outdated
Show resolved
Hide resolved
java/openmldb-batch/src/main/scala/com/_4paradigm/openmldb/batch/utils/HybridseUtil.scala
Outdated
Show resolved
Hide resolved
java/openmldb-batch/src/main/scala/com/_4paradigm/openmldb/batch/utils/HybridseUtil.scala
Outdated
Show resolved
Hide resolved
…20100/OpenMLDB into feat/optimize-tidb-data-integration
java/openmldb-batch/src/main/scala/com/_4paradigm/openmldb/batch/utils/HybridseUtil.scala
Outdated
Show resolved
Hide resolved
The error content of cicd is not caused by the current PR commit. Now apply for manual merge. |
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
feature
What is the current behavior? (You can also link to an open issue here)
tispark data type #3808
What is the new behavior (if this is a feature change)?
supports TiDB data types: smallint, int, datetime.
supported functionalities: offline import (soft copy), online import.
online import: Ignore type checking, support asymmetric type conversion.
data export: Ignore type checking, support asymmetric type conversion, for example, unsigned data.