fairyfar commented on PR #1845: URL: https://github.com/apache/cloudberry/pull/1845#issuecomment-4935419269
> I'm very sorry. The test case of ic-resgroup-v2 failed. It's related to this commit. Let me analyze the reasons. The previous commit modified the `GetResGroupIdForRole` function to read user information from SysCache. In concurrent scenarios, if one session drops a user while another session’s SysCache has not yet been refreshed, the latter may still retrieve information for the dropped user. This behavior is incompatible with the design of resource group. Therefore, I re-evaluated the revisions and implemented a minimal fix. This revision releases the CatalogSnapshot (setting `xmin` to `InvalidTransactionId`) immediately after reading the `pg_authid` catalog. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
