JoaoJandre commented on PR #8737:
URL: https://github.com/apache/cloudstack/pull/8737#issuecomment-1983608195

   > @JoaoJandre We can't run explain analyze with delete commands. Adding the 
index also helps reduce the load on database while cleaning up the entries. 
Otherwise the DB will have to go through each record to filter out all the 
entries which can cause spike in I/O as well. I understand that if the 
administrator reduces the retention period, it can cause issues but having the 
index will speed up deletion which runs every minute.
   
   @vishesh92 you are proposing to add an index to improve DELETE operations; 
however, you are presenting results for SELECT operations. Indeed, indexes can 
improve SELECT operations, which only list the data. However, the DELETE 
operation works different: it removes data, recalculates statistics, 
reorganizes indexes and more. Please, present some tests and results for DELETE 
operations; otherwise, it is not feasible to sustain the reason for adding 
those indexes.
   


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

Reply via email to