[Feature-15146][dolphinscheduler-task-sqoop] add sqoop source/target type #15146
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 add alert plugin aliyun-voice
# Conflicts: # dolphinscheduler-task-plugin/dolphinscheduler-task-sqoop/src/main/java/org/apache/dolphinscheduler/plugin/task/sqoop/SqoopTask.java # dolphinscheduler-ui/src/views/projects/task/components/node/format-data.ts
# Conflicts: # dolphinscheduler-bom/pom.xml # dolphinscheduler-dao/src/main/resources/org/apache/dolphinscheduler/dao/mapper/AlertSendStatusMapper.xml # dolphinscheduler-task-plugin/dolphinscheduler-task-sqoop/src/main/java/org/apache/dolphinscheduler/plugin/task/sqoop/generator/sources/MySQLSourceGenerator.java
xujiaqiang
requested review from
songjianet,
Amy0104,
caishunfeng,
SbloodyS and
zhuangchong
as code owners
xujiaqiang
changed the title
SbloodyS
added
feature
songjianet
previously approved these changes
Nov 13, 2023
songjianet
dismissed
their stale review
Please add the license header to the winning file
xujiaqiang
force-pushed
the
dev-xjq-sqoop
branch
from
0b491ae
to
6096c58
Compare
|
|
I'm not sure whether this hana jar license can use or not. the license of apache compatible license https://apache.org/legal/resolved.html#highlevel |
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #15146 +/- ##
============================================
- Coverage 38.27% 37.80% -0.47%
- Complexity 4694 4701 +7
============================================
Files 1289 1304 +15
Lines 44642 45219 +577
Branches 4787 4859 +72
============================================
+ Hits 17088 17097 +9
- Misses 25691 26259 +568
Partials 1863 1863 ☔ View full report in Codecov by Sentry. |
# Conflicts: # dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-hana/pom.xml
# Conflicts: # dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-all/pom.xml
|
Wei-Alwayers
pushed a commit
to Wei-Alwayers/dolphinscheduler
that referenced
this pull request
Dec 15, 2023
fix typo bug in JSONUtils.java add DBType dolphindb spotless apply & modify ui fix mvn package OK TO DO: Modify DolphinDBDataSourceProcessor fix datasource bug -> connect OK [Improvement-15260][dolphinscheduler-datasource-hana] add hana related dependencies (apache#15260) * add hana related dependencies * optimizing HANA data source bugs * run mvn spotless:apply * Fix hana datasource getValidationQuery() * Fix hana datasource testGetJdbcUrl() --------- Co-authored-by: xujiaqiang <“xujiaqiangwz@163.com”> Co-authored-by: David Zollo <dailidong66@gmail.com> [Feature-15248][dolphinscheduler-alert-plugins] add alert plugin aliyun-voice (apache#15248) * add alert aliyun-voice * VoiceParam replace these with lombok @DaTa * security.ts add alarm_instance params * optimize alert aliyun vocie code * Merge Code * rollback pnpm-lock.yaml to branch :origin/dev version * add the doc about the new plugin * add com.aliyun.dyvmsapi20170525 version 2.14 * completed * run mvn spotless:apply * Code optimization * Change to @DaTa * Code specification optimization * add com.aliyund.yvmsapi20170525.jar license * modify com.aliyund.yvmsapi20170525.jar license * modify com.aliyund.yvmsapi20170525.jar license * add aliyun-vocie licenses * add aliyun-voice link to docsdev.js * modify vocie module to aliyunVoice * add lisense --------- Co-authored-by: xujiaqiang <“xujiaqiangwz@163.com”> Co-authored-by: 旺阳 <wang@lqwang.net> Co-authored-by: David Zollo <dailidong66@gmail.com> [Feature-15146][dolphinscheduler-task-sqoop] add sqoop source/target type (apache#15146) * task list: sgoop node params optimize * security.ts add alarm_instance params * 1 add SqoopTask params 2 add alert plugin aliyun-voice * add license header * commit sqhoop optimize * pnpm-locl.yaml supplement annotation * remove irrelevent commit. * Code specification optimization * optimize sqoop task ui * Merge Code * add the license header to pnpm-locl.yaml * format the code * format the code * Fix sqoop task echo error --------- Co-authored-by: xujiaqiang <xujiaqiang@aimatech.com> Co-authored-by: xujiaqiang <“xujiaqiangwz@163.com”> Co-authored-by: David Zollo <dailidong66@gmail.com> finishgit add dolphinscheduler-common/src/main/java/org/apache/dolphinscheduler/common/constants/DataSourceConstants.java Fix k8sTaskExecutionContext setting configYaml (apache#15116) * fixed the issue of obtaining kubeConfig of k8s tasks * removed excess files * removed excess files * formatted the code --------- Co-authored-by: xiangzihao <460888207@qq.com> Co-authored-by: Aaron Wang <wangweirao16@gmail.com>
Wei-Alwayers
pushed a commit
to Wei-Alwayers/dolphinscheduler
that referenced
this pull request
Dec 15, 2023
fix typo bug in JSONUtils.java add DBType dolphindb spotless apply & modify ui fix mvn package OK TO DO: Modify DolphinDBDataSourceProcessor fix datasource bug -> connect OK [Improvement-15260][dolphinscheduler-datasource-hana] add hana related dependencies (apache#15260) * add hana related dependencies * optimizing HANA data source bugs * run mvn spotless:apply * Fix hana datasource getValidationQuery() * Fix hana datasource testGetJdbcUrl() --------- Co-authored-by: xujiaqiang <“xujiaqiangwz@163.com”> Co-authored-by: David Zollo <dailidong66@gmail.com> [Feature-15248][dolphinscheduler-alert-plugins] add alert plugin aliyun-voice (apache#15248) * add alert aliyun-voice * VoiceParam replace these with lombok @DaTa * security.ts add alarm_instance params * optimize alert aliyun vocie code * Merge Code * rollback pnpm-lock.yaml to branch :origin/dev version * add the doc about the new plugin * add com.aliyun.dyvmsapi20170525 version 2.14 * completed * run mvn spotless:apply * Code optimization * Change to @DaTa * Code specification optimization * add com.aliyund.yvmsapi20170525.jar license * modify com.aliyund.yvmsapi20170525.jar license * modify com.aliyund.yvmsapi20170525.jar license * add aliyun-vocie licenses * add aliyun-voice link to docsdev.js * modify vocie module to aliyunVoice * add lisense --------- Co-authored-by: xujiaqiang <“xujiaqiangwz@163.com”> Co-authored-by: 旺阳 <wang@lqwang.net> Co-authored-by: David Zollo <dailidong66@gmail.com> [Feature-15146][dolphinscheduler-task-sqoop] add sqoop source/target type (apache#15146) * task list: sgoop node params optimize * security.ts add alarm_instance params * 1 add SqoopTask params 2 add alert plugin aliyun-voice * add license header * commit sqhoop optimize * pnpm-locl.yaml supplement annotation * remove irrelevent commit. * Code specification optimization * optimize sqoop task ui * Merge Code * add the license header to pnpm-locl.yaml * format the code * format the code * Fix sqoop task echo error --------- Co-authored-by: xujiaqiang <xujiaqiang@aimatech.com> Co-authored-by: xujiaqiang <“xujiaqiangwz@163.com”> Co-authored-by: David Zollo <dailidong66@gmail.com> finishgit add dolphinscheduler-common/src/main/java/org/apache/dolphinscheduler/common/constants/DataSourceConstants.java Fix k8sTaskExecutionContext setting configYaml (apache#15116) * fixed the issue of obtaining kubeConfig of k8s tasks * removed excess files * removed excess files * formatted the code --------- Co-authored-by: xiangzihao <460888207@qq.com> Co-authored-by: Aaron Wang <wangweirao16@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of the pull request
Brief change log
Verify this pull request
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(or)
If your pull request contain incompatible change, you should also add it to
docs/docs/en/guide/upgrede/incompatible.md