medkhabt opened a new pull request #5556: URL: https://github.com/apache/dolphinscheduler/pull/5556
<!--Thanks very much for contributing to Apache DolphinScheduler. Please review https://dolphinscheduler.apache.org/en-us/community/development/pull-request.html before opening a pull request.--> Hello, this is my first contribution on github, I hope this is the solution you asked for. ## Purpose of the pull request <!--(For example: This pull request adds checkstyle plugin).--> Issue: #5549 Adding null checks in "printReplacedSql" method, for "sqlParamsMap" to make sure there is no NullPointerException thrown. ## Brief change log <!--*(for example:)* - *Add maven-checkstyle-plugin to root pom.xml* --> ## Verify this pull request <!--*(Please pick either of the following options)*--> This change added tests and can be verified as follows: org.apache.dolphinscheduler.server.worker.task.sql.shouldntThrowIllegalArgument_When_SqlParamsMapIsNullOrItsGet_printReplacedSql() <!--*(example:)* - *Added dolphinscheduler-dao tests for end-to-end.* - *Added CronUtilsTest to verify the change.* - *Manually verified the change by testing locally.* --> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
