Skip to content

[Sqoop-Update]modify sqoop task param when upgrade ds to 1.3.4#4212

Merged
xingchun-chen merged 17 commits intoapache:1.3.4-preparefrom
Eights-Li:1.3.4-prepare-sqoop
Dec 12, 2020
Merged

[Sqoop-Update]modify sqoop task param when upgrade ds to 1.3.4#4212
xingchun-chen merged 17 commits intoapache:1.3.4-preparefrom
Eights-Li:1.3.4-prepare-sqoop

Conversation

@Eights-Li
Copy link
Copy Markdown
Contributor

What is the purpose of the pull request

1 add --target-dir param to deal with import mysql to hive using sql query failed
2 modify sqoop task param when ds < 1.3.4 upgrade to 1.3.4
3 fix sqoop task import mysql to hive the web no hive selector

Brief change log

  • add --target-dir param to deal with import mysql to hive using sql query failed
  • modify sqoop task param when ds < 1.3.4 upgrade to 1.3.4
  • fix sqoop task import mysql to hive the web no hive selector

Verify this pull request

This change added tests and can be verified as follows:

  • worker/task/SqoopTaskTest

Eights-Li and others added 17 commits December 6, 2020 12:27
* fix #4043, sqoop import query fail

* fix #4043, sqoop task hard code & code style

* add license for SqoopConstants

* add private constructor for SqoopConstants

* fixed sqoop mysql pwd have special character

* fix checkstyle

* fix sqoop task log

* remove unused constants
* fix #4043, sqoop import query fail

* fix #4043, sqoop task hard code & code style

* add license for SqoopConstants

* add private constructor for SqoopConstants

* fixed sqoop mysql pwd have special character

* fix checkstyle

* fix sqoop task log

* remove unused constants
@sonarqubecloud
Copy link
Copy Markdown

@lgcareer
Copy link
Copy Markdown
Contributor

@Eights-Li I saw the ddl or dml under 1.3.4 is empty,so if 1.3.4 vresion didn't changed table schema or change table field values,so it's better not to create the 1.3.4_schema folder under sql/upgreade.

@xingchun-chen xingchun-chen added this to the 1.3.4-release milestone Dec 12, 2020
@Eights-Li
Copy link
Copy Markdown
Contributor Author

@Eights-Li I saw the ddl or dml under 1.3.4 is empty,so if 1.3.4 vresion didn't changed table schema or change table field values,so it's better not to create the 1.3.4_schema folder under sql/upgreade.

In ds-1.3.4, sqoop task meta data need to upgrade, when running upgrade code, need the schemaversion = 1.3.4, so I add the 1.3.4-schema dir in sql/update

@lgcareer
Copy link
Copy Markdown
Contributor

@Eights-Li I saw the ddl or dml under 1.3.4 is empty,so if 1.3.4 vresion didn't changed table schema or change table field values,so it's better not to create the 1.3.4_schema folder under sql/upgreade.

In ds-1.3.4, sqoop task meta data need to upgrade, when running upgrade code, need the schemaversion = 1.3.4, so I add the 1.3.4-schema dir in sql/update

OK,Got it.

Copy link
Copy Markdown
Contributor

@lgcareer lgcareer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@xingchun-chen
Copy link
Copy Markdown
Contributor

+1

@xingchun-chen xingchun-chen merged commit 933eac7 into apache:1.3.4-prepare Dec 12, 2020
@Eights-Li Eights-Li deleted the 1.3.4-prepare-sqoop branch March 10, 2021 08:06
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.

4 participants