merge with entity_name

Issue #543 resolved
Mike Bayer repo owner created an issue

need to define the semantics of how merge() works given transient entities with possible entity_names.

Comments (8)

  1. Mike Bayer reporter

    OK, changeset:2512 considers entity_name passed to merge() as overriding. also adds entity_name to relation() to make creating mappers easier, and has an attempt at merging fully while avoiding loops, but this logic needs more testing. the first test script also was merging to sqlite0 unnecessarily, so improved test will be attached.

    we also need to put a check for "merging an object thats already in the session", which isnt here and would better detect those errors.

  2. Mike Bayer reporter

    for 0.4.xx, determine if current merge() functionality makes sense, ensure unit test coverage, then close. the user here was trying to do sharding via entity_name so that use case shouldnt be considered for this ticket.

  3. Log in to comment