Fix workflow instance restart failed due to duplicate key in varpool

[ruanwenjun]

Purpose of the pull request

• Fix workflow instance restart failed when using varpool, since duplicate key exist in varpool with (Direct: OUT, IN)

• Unify the direct to OUT in varpool, since it shouldn't be IN, only OUT will localparam will be put into varpool.
• After this PR the varpool will only exist one record for each key.

Brief change log

• Fix the varpool merge logic will cause duplicate key in varpool.
• Fix delete taskInstance failed due to delete log error.
• Load the plugin when server startup, fix the load plugin log will exist in task instance log.

Verify this pull request

Verify by manual.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 52.47525% with 48 lines in your changes are missing coverage. Please review.

Project coverage is 40.58%. Comparing base (68a1906) to head (167e734).

❗ Current head 167e734 differs from pull request most recent head 05d4f3c. Consider uploading reports for the commit 05d4f3c to get more accurate results

Files	Patch %	Lines
...nscheduler/plugin/task/api/utils/VarPoolUtils.java	55.31%	18 Missing and 3 partials ⚠️
.../server/master/runner/WorkflowExecuteRunnable.java	60.86%	8 Missing and 1 partial ⚠️
...plugin/task/api/parameters/AbstractParameters.java	22.22%	6 Missing and 1 partial ⚠️
...uler/api/service/impl/TaskInstanceServiceImpl.java	0.00%	6 Missing ⚠️
...source/api/plugin/DataSourceProcessorProvider.java	0.00%	2 Missing ⚠️
...e/dolphinscheduler/server/master/MasterServer.java	0.00%	1 Missing ⚠️
...uler/plugin/task/api/parameters/SqlParameters.java	50.00%	1 Missing ⚠️
...e/dolphinscheduler/server/worker/WorkerServer.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##                dev   #16001      +/-   ##
============================================
+ Coverage     40.51%   40.58%   +0.07%     
- Complexity     5198     5216      +18     
============================================
  Files          1379     1380       +1     
  Lines         46116    46056      -60     
  Branches       4931     4916      -15     
============================================
+ Hits          18682    18691       +9     
+ Misses        25505    25439      -66     
+ Partials       1929     1926       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[fuchanghai]

LGTM ,if CI pass

[Radeity]

LGTM

[rickchengx]

LGTM

[sonarcloud]

Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
61.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud