[Checkins] [zopefoundation/ZODB] f2922e: When creating a "new_instance", copy loadBefore, a...

GitHub noreply at github.com
Wed May 4 22:13:55 CEST 2016


  Branch: refs/heads/no-more-load
  Home:   https://github.com/zopefoundation/ZODB
  Commit: f2922e4cc6ea5ca4d7e3ba0f18c970485c1d3c8b
      https://github.com/zopefoundation/ZODB/commit/f2922e4cc6ea5ca4d7e3ba0f18c970485c1d3c8b
  Author: Jim Fulton <jim at zope.com>
  Date:   2016-05-04 (Wed, 04 May 2016)

  Changed paths:
    M src/ZODB/tests/MVCCMappingStorage.py

  Log Message:
  -----------
  When creating a "new_instance", copy loadBefore, as well s pack.

This is needed because pack temporarily deletes data from the data
structure that loadBefore reads.

Note that in the new scheme of things, we avoid using load, but
instead use loadBefore.  This we bypass the _data_snapshot
shenanigans, which makes me think we aren't testing what we should be,
but I have no confidence that the machinery in MVCCMappingStorage
actually reflects anything meaningful.




More information about the checkins mailing list