Merge pull request #5667 from BigRoy/bugfix/publisher_reset_report_maker_memory_leak

Publisher: Fix report maker memory leak + optimize lookups using set
This commit is contained in:
Jakub Trllo 2023-10-01 22:56:07 +02:00 committed by GitHub
commit baf6945b4d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -179,7 +179,7 @@ class PublishReportMaker:
self._plugin_data = []
self._plugin_data_with_plugin = []
self._stored_plugins = []
self._stored_plugins = set()
self._current_plugin_data = []
self._all_instances_by_id = {}
self._current_context = None
@ -194,6 +194,7 @@ class PublishReportMaker:
self._publish_discover_result = create_context.publish_discover_result
self._plugin_data = []
self._plugin_data_with_plugin = []
self._stored_plugins = set()
self._current_plugin_data = {}
self._all_instances_by_id = {}
self._current_context = context
@ -229,7 +230,7 @@ class PublishReportMaker:
raise ValueError(
"Plugin '{}' is already stored".format(str(plugin)))
self._stored_plugins.append(plugin)
self._stored_plugins.add(plugin)
plugin_data_item = self._create_plugin_data_item(plugin)