Project

Profile

Help

Issue #3903

Redundant solver instantiated and loaded during a recursive associate call

Added by milan 11 months ago. Updated 2 months 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 11 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 11 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 11 months ago

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

#3 Updated by milan 11 months ago

  • Status changed from POST to MODIFIED

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

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

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

  • Status changed from MODIFIED to ON_QA

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

  • Status changed from ON_QA to CLOSED - CURRENTRELEASE

#7 Updated by bmbouter 2 months ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF