#203 add sqoop task#1974
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #1974 +/- ##
============================================
+ Coverage 30.84% 31.96% +1.12%
- Complexity 1382 1511 +129
============================================
Files 294 311 +17
Lines 15022 15402 +380
Branches 1810 1864 +54
============================================
+ Hits 4633 4923 +290
- Misses 9930 9969 +39
- Partials 459 510 +51
Continue to review full report at Codecov.
|
|
1 add license |
| /** | ||
| * sqoop task extends the shell task | ||
| */ | ||
| public class SqoopTask extends ShellTask { |
There was a problem hiding this comment.
sqoop tasks will be converted to mapreduce tasks, managed by yarn, SqoopTask class need extends AbstractYarnTask not ShellTask
|
SonarCloud Quality Gate failed.
|
|
please make sure code have been checked by ali style plugin. |
| import java.util.List; | ||
|
|
||
| /** | ||
| * @author simfo |
There was a problem hiding this comment.
remove the author and time
|
|
||
| // StringBuilder srcWhere = new StringBuilder(); | ||
| // if(sourceMysqlParameter.getSrcConditionList().size()>0){ | ||
| // List<Property> items = sourceMysqlParameter.getSrcConditionList(); |
There was a problem hiding this comment.
remove this comment out code
|
@break60 please review front-end~ |
|
+1 |
|
Kudos, SonarCloud Quality Gate passed!
|
What is the purpose of the pull request
This pull request adds sqoop task.
Brief change log
Verify this pull request
This change added tests and can be verified as follows: