The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- Status changed to Needs review
almost 2 years ago 1:00am 31 January 2023 - Assigned to bhanu951
- @bhanu951 opened merge request.
- 🇧🇪Belgium borisson_ Mechelen, 🇧🇪
> Does it require BC Shim ? as this is not a public service.
This is a really good question. I think that we don't need the BC shim because of https://www.drupal.org/about/core/policies/core-change-policies/bc-polic... → .
- 🇬🇧United Kingdom catch
I re-read the discussion from five years ago, and I think if we end up with a working and viable patch here for the original approach, we should probably split the cache collector idea out to its own issue - that might be even better, but needs thought on exactly where to put it and how it would work.
- Status changed to Needs work
almost 2 years ago 1:35am 11 February 2023 - 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
There are some open threads and a test failure in the MR.
- 🇺🇸United States smustgrave
Closed 🐛 "non admin routes" cache fragmentation Closed: duplicate as a duplicate
