Skip to content

Make Direct Transfer logging more consistent

James Nutt requested to merge jnutt/improve-dt-logs into master

What does this MR do and why?

Make Direct Transfer logging more consistent

This change attempts to satisfy the following issue:

Follow-up from "Improve Direct Transfer logs" #430880 (closed)

It allows us to associate a BulkImport::Logger instance with an entity, creating a more user friendly API for consistent logging across the Direct Transfer code.

The initial issue suggested simplifying the logging API for objects other than entity, but as far as I can see, entity is the only one used consistently across users of the BulkImports::Logger. I think it makes sense to set the precedent in this MR but leave other models until we have more use-cases to abstract from.

Changelog: other

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by James Nutt

Merge request reports