[GitHub] bjustin-ibm opened a new pull request #198: Do not attempt to share DB connections across multiple processes

[gi...@git.apache.org]

bjustin-ibm opened a new pull request #198: Do not attempt to share DB connections across multiple processes
URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/198
 
 
   At the moment, this also includes changes from #197. Please ignore these commits here as they will go away once 197 is merged.
   
   The problem is that keeping the DB connection as a class property causes it to be re-used, potentially between multiple processes. For SSL connections in Python, this is not supported.
   
   The fix is to ensure that the DB connection is made an instance field of the Database class, and also to ensure that all Database instances are themselves instance fields of their respective classes. This avoids sharing trying to share the same connection between multiple Python processes.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services