- Issue created by @jienckebd
- Status changed to Closed: works as designed
9 months ago 2:03pm 12 March 2024 - π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
So I'm thinking we run dblog_cron() whenever a migration starts to delete log messages based on the "row_limit" config key of the "dblog.settings" config.
-1. This would slow down the migration drastically.
The proper solution is to contribute patches to prevent those deprecation errors from being triggered in the first place, and adding them to the recommendations, i.e. to https://git.drupalcode.org/project/acquia_migrate/-/blob/recommendations....
- Status changed to Postponed: needs info
9 months ago 5:56pm 12 March 2024 - π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Did π Fix deprecation error: Calling Sql::__construct() without the $migration_manager Fixed and π Fix deprecation error: Calling Migration::getMigrationDependencies() without expanding the plugin IDs Fixed help reduce this problem?
- πΊπΈUnited States jienckebd
-1. Drastic slow down is exaggerated. A few SQL queries per migration import task is < 0.1% impact. Total cost / benefit -- that would save our team time tracking down logging issues across all of Drupal.
Proper solution in a perfect world is as you describe, but we're balancing other constraints.
- Status changed to Closed: works as designed
8 months ago 4:07pm 8 April 2024