diff options
| author | Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com> | 2015-09-23 13:39:27 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-09-26 18:03:29 +0100 |
| commit | c1b4754f69003df1a83fafc1c80a9ef74400b6dd (patch) | |
| tree | 03beaca4aad090bbfbd0653bc19b74021f39fcf7 | |
| parent | bd6398e967c234e89d773f509512ebf460fa76ff (diff) | |
| download | bitbake-c1b4754f69003df1a83fafc1c80a9ef74400b6dd.zip | |
prserv/serv: Close the DB connection out of class destructor
When launching the PR server daemon, the PRData __del__ function was being
called (no reason found yet) where the DB connection closed, thus following
PR updates were not getting into the DB. This patch closes the connection
explicitly, not relaying on the __del__ function execution.
Closing the connection in turn causes all WAL file transactions to be moved
into the database (checkpoint), thus effectively updating the database.
[YOCTO #8215]
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | lib/prserv/db.py | 2 | ||||
| -rw-r--r-- | lib/prserv/serv.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/lib/prserv/db.py b/lib/prserv/db.py index 43795801..36c9f7b6 100644 --- a/lib/prserv/db.py +++ b/lib/prserv/db.py @@ -248,7 +248,7 @@ class PRData(object): self.connection.execute("PRAGMA journal_mode = WAL;") self._tables={} - def __del__(self): + def disconnect(self): self.connection.close() def __getitem__(self,tblname): diff --git a/lib/prserv/serv.py b/lib/prserv/serv.py index 3cb2e03d..c557837b 100644 --- a/lib/prserv/serv.py +++ b/lib/prserv/serv.py @@ -148,7 +148,7 @@ class PRServer(SimpleXMLRPCServer): while not self.quit: self.handle_request() self.handlerthread.join() - self.table.sync() + self.db.disconnect() logger.info("PRServer: stopping...") self.server_close() return |