Merge pull request #11 from gmjosack/dumb-gary

Fix bug in new multiprocessing lazy loading
dropbox · Jun 17, 2015 · 6b865f5 · 6b865f5
2 parents 135509c + 0cc62a7
commit 6b865f5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/hermes/handlers/util.py b/hermes/handlers/util.py
@@ -36,12 +36,13 @@ def initialize(self):
         my_settings = self.application.my_settings
 
         # Lazily build the engine and session for support of multi-processing
-        if my_settings.get("db_engine") is None and my_settings.get("db_session") is None:
+        if my_settings.get("db_engine") is None:
             my_settings["db_engine"] = models.get_db_engine(my_settings.get("db_uri"))
-            my_settings["db_session"] = models.Session.configure(bind=my_settings["db_engine"])
+            models.Session.configure(bind=my_settings["db_engine"])
+            my_settings["db_session"] = models.Session
 
-        self.session = my_settings.get("db_session")()
         self.engine = my_settings.get("db_engine")
+        self.session = my_settings.get("db_session")()
         self.domain = my_settings.get("domain")
 
     def on_finish(self):

diff --git a/hermes/version.py b/hermes/version.py
@@ -1 +1 @@
-__version__ = "0.1.19"
+__version__ = "0.1.20"