freshmaker

#298 Cache ODCS pulp composes with the same content_sets in record_batches.
Merged by jkaluza. Opened by jkaluza.
jkaluza/freshmaker cache-pulp-compose  into  master

Download 298.patch

For advisories with lot of images, we end up asking for the same ODCS pulp compose again and again. Instead, we can just reuse the old compose.

+1

Suggestion: Why not just save it to cache just after call to self._prepare_pulp_repo?

Can this db_compose be cached as well with associated pulp compose together? It can save cost of a few SELECT queries.
Edited by cqi

Looks good for me generally. Two minor suggestions.

1 new commit added

  • Cache the Compose database object instead of ODCS compose dict.

@cqi: I've updated PR to address your comments, please check :).

Looks good for me. :thumbsup:

Commit a755b24b fixes this pull-request

Pull-Request has been merged by jkaluza

Pull-Request has been merged by jkaluza
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.