[fix][python] Correct variable type of SQL type

[hiSandog]

When SQL type tasks are uploaded through Python-Gate, the web page displays an error.

If I create sql type task from web page, the type of sql-type is string.
But when I upload SQL type tasks via Python-Gate, the type of sql-type is int.
So these two sides should be unified.

fix: #10463

[SbloodyS]

Hi @hiSandog ,please associate issue.

[zhongjiajie]

When SQL type tasks are uploaded through Python-Gate, the web page displays an error.

If I create sql type task from web page, the type of sql-type is string. But when I upload SQL type tasks via Python-Gate, the type of sql-type is int. So these two sides should be unified.

#10463

Hi @hiSandog, when you target to a specific issue, you should use the keyword fix or close before it, such as fix: #10463 in this case, and the issue will be closed when the PR merged

[zhongjiajie]

FYI @SbloodyS I change the milestone from 300b3 to 300b2 due to this is a bugfix and we still not close the 300b2 bug fix process

[zhongjiajie]

I wonder why this will not fail our tests, and go to see our SQL task tests. finally I find we just using the variable instead of constants. 🤣 @hiSandog please change

dolphinscheduler/dolphinscheduler-python/pydolphinscheduler/tests/tasks/test_sql.py

Line 141 in 720a10a

"sqlType": SqlType.SELECT,

this line from SqlType.SELECT to "1", using the contants variable in uinttest is a bad choose

[zhongjiajie]

and you can change the function test_property_task_params too if you want. The test_get_sql_type can or can not change because if only test the sql type itself instead of the final param pass to python gateway server

[zhongjiajie]

LGTM, waiting for CI

[codecov-commenter]

Codecov Report

Merging #10464 (9576956) into dev (eecc49e) will decrease coverage by 0.00%.
The diff coverage is 60.00%.

@@             Coverage Diff              @@
##                dev   #10464      +/-   ##
============================================
- Coverage     40.90%   40.89%   -0.01%     
- Complexity     4850     4851       +1     
============================================
  Files           883      883              
  Lines         35970    35972       +2     
  Branches       3991     3991              
============================================
- Hits          14712    14711       -1     
- Misses        19802    19806       +4     
+ Partials       1456     1455       -1     

Impacted Files	Coverage Δ
...inscheduler/api/service/impl/UsersServiceImpl.java	72.50% <0.00%> (ø)
.../org/apache/dolphinscheduler/common/Constants.java	80.95% <ø> (ø)
.../org/apache/dolphinscheduler/api/enums/Status.java	100.00% <100.00%> (ø)
.../apache/dolphinscheduler/api/utils/CheckUtils.java	75.86% <100.00%> (+0.86%)	⬆️
...e/dolphinscheduler/remote/NettyRemotingClient.java	50.00% <0.00%> (-2.78%)	⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java	53.33% <0.00%> (ø)
...er/master/dispatch/host/assign/RandomSelector.java	83.33% <0.00%> (+5.55%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d16e0dd...9576956. Read the comment docs.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[zhongjiajie]

LGTM, thanks to bring this up