zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

performance: avoid repeated DB locking on view generation

Browse Source 
`ViewDescriptor.regenerate()` checks repeatedly whether packages are
installed and also does a lot of DB queries.  Put a read transaction
around the whole thing to avoid repeatedly locking and unlocking the DB.
This commit is contained in:
Todd Gamblin 2019-12-20 23:38:23 -08:00
parent 91ea90c253
commit a85b9070cb
No known key found for this signature in database
GPG Key ID: 66B24B9050FDD0B8
1 changed files with 13 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/spack/spack/environment.py
View File

@ -516,8 +516,11 @@ def regenerate(self, all_specs, roots):
if spec.concrete: # Do not link unconcretized roots if spec.concrete: # Do not link unconcretized roots
specs_for_view.append(spec.copy(deps=('link', 'run'))) specs_for_view.append(spec.copy(deps=('link', 'run')))
installed_specs_for_view = set(s for s in specs_for_view # regeneration queries the database quite a bit; this read
if s in self and s.package.installed) # transaction ensures that we don't repeatedly lock/unlock.
with spack.store.db.read_transaction():
installed_specs_for_view = set(
s for s in specs_for_view if s in self and s.package.installed)
view = self.view() view = self.view()