Project

Profile

Help

Issue #3903

Redundant solver instantiated and loaded during a recursive associate call

Added by milan almost 3 years ago. Updated about 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
2.17.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 41
Quarter:

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 almost 3 years 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 almost 3 years 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 almost 3 years ago

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

#3 Updated by milan almost 3 years ago

  • Status changed from POST to MODIFIED

#4 Updated by ipanova@redhat.com almost 3 years ago

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

#5 Updated by ipanova@redhat.com over 2 years ago

  • Status changed from MODIFIED to 5

#6 Updated by ipanova@redhat.com over 2 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#7 Updated by bmbouter about 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF