Project

Profile

Help

Issue #3903

Redundant solver instantiated and loaded during a recursive associate call

Added by milan 9 months ago. Updated 5 days ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Severity:
2. Medium
Version:
Platform Release:
2.17.0
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 41

Description

It seems a solver is being unnecessarily instantiated and loaded in a recursive associate call
This wastes a lot of RAM and CPU with big repositories and might be avoided.

Associated revisions

Revision 3a42248d View on GitHub
Added by milan 9 months ago

Share solver between recursive associate calls

Currently each associate call in a recursive tree of calls instantiates
and loads its own instance of a solver. This is both CPU and RAM
expensive.

The fix avoids subsequent instantiation of the solver thru passing it as
an argument of the associate function.

Fixes: #3903

History

#1 Updated by milan 9 months ago

  • Status changed from NEW to POST
  • Assignee set to milan

Just created a PR to address this: https://pulp.plan.io/issues/3903

#2 Updated by CodeHeeler 9 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 41

#3 Updated by milan 9 months ago

  • Status changed from POST to MODIFIED

#4 Updated by ipanova@redhat.com 9 months ago

  • Sprint/Milestone set to 2.17.0
  • Platform Release set to 2.17.0

#5 Updated by ipanova@redhat.com 8 months ago

  • Status changed from MODIFIED to ON_QA

#6 Updated by ipanova@redhat.com 8 months ago

  • Status changed from ON_QA to CLOSED - CURRENTRELEASE

#7 Updated by bmbouter 5 days ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF