Problem/Motivation

Since APCu might already be available, it could be the easiest option, whereas installing Drupal Redis requires a few steps.

Steps to reproduce

Proposed resolution

Clarify that APCu is probably the easiest option, by adding something like this in the README?

You may already have APCu installed. Check PHP status and /admin/reports/status for these lines:

PHP APCu available caching: Memory available: 98.23 MB.
PHP APCu caching: Enabled (128 MB)

To give APCu more memory, add this in /etc/php/8.4/apache2/php.ini (for PHP 8.4) and restart the server:

# More memory for APCu
apc.shm_size = 256M

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork crawler_rate_limit-3552344

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

ressa created an issue. See original summary.

vaish’s picture

vaish commented

Thanks for the suggestion. Could you please create a MR. I would just avoid providing a full path to php.ini since that path depends on the server configuration. Better just to mention the file name (php.ini) and to rely on the user to know how to locate the file on their server.

ressa opened merge request !13

ressa’s picture

ressa
he/him
commented
Status: Active » Needs review

Great suggestion to simplify the path, since like you write, the user probably knows the location.

ressa’s picture

ressa
he/him
commented

Wouldn't it be great to get this pushed over the finishing line? :)

vaish’s picture

vaish commented

Hello, @ressa. I apologize for delay. I'm back now. I made some further changes to the text. Please check it out and let me know if it's ok with you.

ressa’s picture

ressa
he/him
commented
Status: Needs review » Reviewed & tested by the community

No problem, thanks for taking a look at it and suggesting some changes. Though I don't see any advice on how to chose optimal amount of APCu memory via that link ... I added the "# More memory for APCu -- apc.shm_size = 256M" example as a quick start, so the user can have a starting point, and adjust up or down as they see fit. But of course, linking to the documentation also works, though it may take a little longer for the user to complete the task.

  • vaish committed 8e01a318 on 3.x authored by ressa 
    docs: #3552344 Clarify that APCu may already be available

By: ressa
By...
vaish’s picture

vaish commented
Status: Reviewed & tested by the community » Fixed

Thank you @ressa. I added back the configuration example.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

ressa’s picture

ressa
he/him
commented

It looks good, thanks @vaish!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.