mlsorensen commented on PR #8737: URL: https://github.com/apache/cloudstack/pull/8737#issuecomment-1994732867
Yes I think we can all agree that the big one here is index on vm_id. Missing this is absolutely crushing for default listVirtualMachines calls (where stats are included). Maybe we can at least get that index in place and discuss further on whether or not the timestamp index is beneficial overall. FWIW I don't see a problem with an odd deletion of stats taking a long time occasionally, if an index improves the majority use cases. However, if it's so slow to delete that it is impossible to do so, or breaks something, then that needs to be resolved, maybe by batch deletion or something. -- 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: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
