Skip to content
This repository has been archived by the owner. It is now read-only.

Commit after update #2

Merged
merged 2 commits into from Apr 28, 2015
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Next

Added transaction commiting after the SQL update

It appears that the transaction would need to be managed externally
in order to actually update the DB.  Now do it at the end of the
bulk_update call.
  • Loading branch information
Scott Griffin
Scott Griffin committed Apr 28, 2015
commit 7bcd6978b09c9b132d1843ccf2ad0bb8ee23bc88
@@ -139,3 +139,4 @@ def bulk_update(objs, meta=None, update_fields=None, exclude_fields=None,
del values, pks

connection.cursor().execute(sql, parameters)
transaction.commit_unless_managed(using=using)
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.