Problem/Motivation

Follow-up from #2503999-32: Large volume entity migrations run out of memory
Core maintainer comment https://www.drupal.org/node/2503999#comment-10266631

+++ b/core/modules/migrate/src/MigrateExecutable.php
@@ -568,7 +568,17 @@ protected function attemptMemoryReclaim() {
+    // Entity storage can blow up with caches so clear them out.
+    $container = \Drupal::getContainer();
+    /** @var \Drupal\Core\Entity\EntityManagerInterface $manager */
+    $manager = $container->get('entity.manager');

There's \Drupal::entityManager() for that

Proposed resolution

Use \Drupal::entityManager()

Remaining tasks

User interface changes

API changes

Data model changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andriyun created an issue. See original summary.

andriyun’s picture

Status: Active » Needs review
FileSize
732 bytes
andriyun’s picture

Issue summary: View changes
andypost’s picture

Component: entity system » migration system
Assigned: andriyun » Unassigned
Status: Needs review » Reviewed & tested by the community
Issue tags: -Entity Field API, -Migrate critical +Quick fix, +Quickfix

thanx for filing new issue

mikeryan’s picture

Priority: Major » Minor
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, that's a nice clean-up!

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 2d5dc23 on 8.0.x
    Issue #2559191 by andriyun, andypost: Clean-up migrate executable
    

Status: Fixed » Closed (fixed)

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