Project

Profile

Help

Issue #3903

closed

Redundant solver instantiated and loaded during a recursive associate call

Added by milan almost 6 years ago. Updated over 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.

Actions #1

Updated by milan almost 6 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

Actions #2

Updated by CodeHeeler almost 6 years ago

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

Added by milan almost 6 years ago

Revision 3a42248d | View on GitHub

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

Actions #3

Updated by milan almost 6 years ago

  • Status changed from POST to MODIFIED
Actions #4

Updated by ipanova@redhat.com almost 6 years ago

  • Sprint/Milestone set to 2.17.0
  • Platform Release set to 2.17.0
Actions #5

Updated by ipanova@redhat.com almost 6 years ago

  • Status changed from MODIFIED to 5
Actions #6

Updated by ipanova@redhat.com almost 6 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #7

Updated by bmbouter over 5 years ago

  • Tags Pulp 2 added
Actions #8

Updated by ttereshc over 2 years ago

  • Sprint/Milestone deleted (2.17.0)

Also available in: Atom PDF