[Checkins] [zopefoundation/ZODB] 1b9475: Review Connection/DB closure

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/zopefoundation/ZODB
  Commit: 1b9475d413705e510ff2f19ee6449688ffb04d46
      https://github.com/zopefoundation/ZODB/commit/1b9475d413705e510ff2f19ee6449688ffb04d46
  Author: Julien Muchembled <jm at nexedi.com>
  Date:   2017-11-08 (Wed, 08 Nov 2017)

  Changed paths:
    M src/ZODB/Connection.py
    M src/ZODB/DB.py

  Log Message:
  -----------
  Review Connection/DB closure

In addition to some micro-optimisation, this fixes the following minor issues:
- Closing a DB left Connections registered to the global transaction manager.
  Which broke at least multi-db because only such primary connections were
  ignored with monkey-patches.
- Stop calling the real newTransaction when aborting an existing transaction.
  This was not the case if explicit_transactions=0.


  Commit: d71f76d8b55b8ada49616b40bcb5e0830d3110b2
      https://github.com/zopefoundation/ZODB/commit/d71f76d8b55b8ada49616b40bcb5e0830d3110b2
  Author: Jim Fulton <jim at jimfulton.info>
  Date:   2017-11-12 (Sun, 12 Nov 2017)

  Changed paths:
    M src/ZODB/DB.py

  Log Message:
  -----------
  Add a comment to explain a slight hack.


  Commit: 87b2654996bd7c92594343e31333868684484ea7
      https://github.com/zopefoundation/ZODB/commit/87b2654996bd7c92594343e31333868684484ea7
  Author: Julien Muchembled <jm at jmuchemb.eu>
  Date:   2017-11-13 (Mon, 13 Nov 2017)

  Changed paths:
    M src/ZODB/Connection.py
    M src/ZODB/DB.py

  Log Message:
  -----------
  Merge pull request #181 from zopefoundation/closure

Review Connection/DB closure


Compare: https://github.com/zopefoundation/ZODB/compare/cb928231e589...87b2654996bd