fedora-infra / mirrormanager2

Rewrite of the MirrorManager application in Flask and SQLAlchemy
https://mirrormanager.fedoraproject.org
GNU General Public License v2.0
65 stars 49 forks source link

Defer loading of host.config #156

Closed adrianreber closed 8 years ago

adrianreber commented 8 years ago

The field host.config is never used but increases the required memory dramatically for certain database operation. Its loading is now deferred and as it is never read the call write the data has also been removed.

adrianreber commented 8 years ago

The MM1 code also does not store the config anywhere. So this change in behaviour has somehow crept into MM2. This is probably a database field which could be removed. Never read, never written.

puiterwijk commented 8 years ago

Looks sane to me. :+1:

pypingou commented 8 years ago

:+1: for me as well

adrianreber commented 8 years ago

Thanks!