BenjaminWenqiYu opened a new pull request #6409:
URL: https://github.com/apache/dolphinscheduler/pull/6409


   …istResourceByUser'
   
   2. ResourceMapper add existResource function without the userId condition
   3. change the ResourceServiceImpl's private function 'checkResourceExists' 
without userId condition
   
   <!--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.-->
   
   
   ## Purpose of the pull request
   
   the private function 'checkResourceExists' should remove the userId 
condition;
   If we want to check the resouce exists or not, we should not user the userId 
condition.
   Because the resource was created by user, it's userId was the user who 
created it.
   If we user the userID '0', It won't take effect.
   
   ## Brief change log
   
   1. ResourceMapper rename the original 'existResource' function to 
'existResourceByUser'
   2. ResourceMapper add existResource function without the userId condition
   3. change the ResourceServiceImpl's private function 'checkResourceExists' 
without userId condition
   ## Verify this pull request
   
   
   This pull request is code cleanup without any test coverage.
   
   resolve #6408 
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to