GaOrtiga commented on PR #12607: URL: https://github.com/apache/cloudstack/pull/12607#issuecomment-4216054380
@SURYAS1306 This approach doesn't really fix the root cause of the problem. When a project is removed, it tries to remove every record of accounts owning it from the DB. The problem happens because on `ProjectAccountDaoImpl.java` the parameter `UserId` is set to null for every query for this table, so when it deletes the project and tries to delete its records its triste matching the `accountid` parameter with the account's id and the `userid` parameter with null, since the record has a non-null `userid`, it does not get removed. The problem with this approach is that it allows theses records to be kept in the DB after the project is removed, which they shouldn't, the right approach would be to make the changes only in the Dao. -- 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]
