flush post-accounting shouldn't operate on expiered instances when autocommit=True

Issue #2041 resolved
Mike Bayer repo owner created an issue

post-flush emits a SELECT for every row otherwise

Comments (5)

  1. Mike Bayer reporter
    • removed status
    • changed status to open

    I think I might want to try lifting the original limitation here, move the flush accounting inside the autocommit transaction .

  2. Mike Bayer reporter

    Moved the post flush as well as the event hook into the try/except block in 8f381f202e569eda0d5de13f584304440492222c. The change to autoflush=False is that the accounting + the hook are inside the scope of where rollback() would be called. There's a potential for backwards incompatible here if people have post_flush_exec hooks that either throw exceptions or call rollback(). Let's see how it goes.

  3. Log in to comment