kyoty opened a new pull request #5670: URL: https://github.com/apache/dolphinscheduler/pull/5670
## Purpose of the pull request Fix a potential problem that editing an existing mysql data source may cause the backend to return null data. reproduce step: 1. Create a normal MySQL data source 2. Modify the IP of the data source created in the step 1, by modify ip to a non-existent IP, and then directly click the edit button (skip the test connection), then we'll seen an error message tip with no content  Request URL: http://localhost:8888/dolphinscheduler/datasources/update response {"code":null,"msg":null,"data":null} ## Brief change log fix a special case in edit an existing datasource. ```java Result<Object> isConnection = checkConnection(dataSource.getType(), connectionParam); if (Status.SUCCESS.getCode() != isConnection.getCode()) { return result; } ``` The result instance does not have a certain code and msg value, which results in the content returned to the front end shows as `nul`l. In this case, just return isConnection directly. ## Verify this pull request Using Existed UTs. Alse munutal test works well -- 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]
