Timeout with a lot of revisions

Created on 25 February 2020, over 4 years ago
Updated 7 March 2024, 4 months ago

We have over a million revisions and this causes troubles to get the candidates revisions for deletion. Unfortunately, the method `getCandidatesRevisions` tries to load all revisions.

This is called when we try to reach the admin page, so because we have 1M revisions, we get a timeout.

Even if we manage to get the module configured to run the cron, we hit the same problem as the same method is called and hits the timeout.

It would be great, I think, to have some hard limit when loading the revisions in order to avoid the timeout.

πŸ› Bug report
Status

Closed: outdated

Version

1.0

Component

Performance

Created by

πŸ‡¨πŸ‡­Switzerland yvmarques

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡³πŸ‡±Netherlands seanB Netherlands

    We just added a 2.x version of the module. This is a complete rewrite based on plugins to determine which revisions can be deleted. It also uses a queue worker to actually delete revisions. Could you please test if this solves the timeout issues you currently experience?

  • πŸ‡³πŸ‡΄Norway steinmb

    We have an older D7 site that also give me grief, 4GB memory is not enough on 7.x-3.2. No timeouts, but run out of memory and is a general performance problem.

    Entering this part consume a lot of memory.

    ```
    module_invoke($module = 'node_revision_delete', $hook = 'cron')

    ```

    To allow cron to finish I had to return early.

    function node_revision_delete_cron() {
      return;
    

    Just to make sure the rest of the cron tasks works as expected.

  • πŸ‡¬πŸ‡ͺGeorgia almador

    what about 28 000 000 rows in node_revision_body and 8 more revision-related tables?

    so I have 9 tables, and every table contains around 30 000 000 records, how to speed up the process and make it work with this module way faster?

  • Status changed to Closed: outdated 4 months ago
  • πŸ‡¨πŸ‡¦Canada adriancid Montreal, Canada

    Closing because the current development is being made in branch 2.x

Production build 0.69.0 2024