Merge pull request #723 from wking/index-db-results-close-session

dmp42 · dmp42 · commit 362b5782c1cb · 2014-11-13T09:06:49.000-08:00
docker_registry.lib.index.db: Close session in SQLAlchemyIndex.results
diff --git a/docker_registry/lib/index/db.py b/docker_registry/lib/index/db.py
@@ -123,9 +123,11 @@ def results(self, search_term=None):
                 sqlalchemy.sql.or_(
                     Repository.name.like(like_term),
                     Repository.description.like(like_term)))
-        return [
+        results = [
             {
                 'name': repo.name,
                 'description': repo.description,
             }
             for repo in repositories]
+        session.close()
+        return results

Original file line number	Diff line number	Diff line change
`@@ -123,9 +123,11 @@ def results(self, search_term=None):`
`123`	`123`	`sqlalchemy.sql.or_(`
`124`	`124`	`Repository.name.like(like_term),`
`125`	`125`	`Repository.description.like(like_term)))`
`126`		`- return [`
	`126`	`+ results = [`
`127`	`127`	`{`
`128`	`128`	`'name': repo.name,`
`129`	`129`	`'description': repo.description,`
`130`	`130`	`}`
`131`	`131`	`for repo in repositories]`
	`132`	`+ session.close()`
	`133`	`+ return results`