[Fix-5549][WorkerServer] SqlTask NullPointerException #5549

[didiaode18]

Purpose of the pull request

This close #5549.

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:

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[ruanwenjun]

@didiaode18
The main problem in this issue #5549 might be that the value in the sqlParamsMap may be null.
And there is a same pr #5556

[CalvinKirs]

Sorry, someone submitted PR before you.
If the original PR has not been updated for a long time (usually two weeks),
we will accept the new PR.

[didiaode18]

Thank you for your reminder, how to deal with the pr that has not been merged under the consultation will not affect my next submission @ruanwenjun @CalvinKirs

[CalvinKirs]

Deeply thanks for your contribution to this.
According to the open-source culture, who submit first, who get merged, if it is good enough.,
Therefore, I will close this PR. I am very sorry for that. I look forward to your continued contribution. If you encounter any problems, please feel free to communicate with me.

[CalvinKirs]

Thank you for your reminder, how to deal with the pr that has not been merged under the consultation will not affect my next submission @ruanwenjun @CalvinKirs

It is best to check if someone has submitted a PR before submitting it.