When content count grows huge, certain pages will start getting performance issues. One of the example is admin/content pages. Since by default the module does a strict check against permission, while listing etc. which introduces major performance loss.

Configuration "Disable node access records" fixes the issue, however its hard to capture that this configuration can possibly cause performance issue.

I would propose if we can change the verbiage to make the performance gain more explicit so that it can easily catch attention.

From
issue_lang

To
fix_lang

Comments

arpitr created an issue. See original summary.

tdnshah’s picture

Assigned: Unassigned » tdnshah
tdnshah’s picture

Status: Active » Needs review
StatusFileSize
new1.27 KB

Updated the help text as mentioned in the issue description.

tdnshah’s picture

Assigned: tdnshah » Unassigned
tdnshah’s picture

StatusFileSize
new1.56 KB
tdnshah’s picture

Version: 8.x-1.x-dev » 8.x-2.12

Updating the issues module version as 8.x-1.x does not have the needed fields for this patch hence the issue cannot reproduced in 8.x-1.x branch. For reproducing the error we need 8.x.2.12 and above branch.

tdnshah’s picture

StatusFileSize
new1.09 KB

Adding the patch for 8.x.2.12 version
8.x-2.x-dev version is not available hence test will fail so not adding the automated test.

jepster_’s picture

Status: Needs review » Fixed

Thanks, I have improved wording slightly and added a bit more information. Also I have added some documentation about cache warming for you: https://www.drupal.org/docs/8/modules/permissions-by-term#s-warmed-page-...

See the PbT release version 8.x-2.23: https://www.drupal.org/project/permissions_by_term/releases/8.x-2.23

jepster_’s picture

Status: Fixed » Closed (fixed)