Homec4science

Restore Lisk transactional methods

Authored by epriestley <git@epriestley.com> on Jan 31 2012, 21:07.

Description

Restore Lisk transactional methods

Summary:
Restores a (simplified and improved) version of Lisk transactions.

This doesn't actually use transactions anywhere yet. DifferentialRevisionEditor
is the #1 (and only?) case where we have transaction problems right now, but
sticking save() inside a transaction unconditionally will leave us holding a
transaction open for like a million years while we run Herald rules, etc. I want
to do some refactoring there separately from this diff before making it
transactional.

NOTE: @jungejason / @nh, can one of you verify these unit tests pass on HPHP/i/vm when you get a chance? I vaguely recall there was some problem with (int)$resource. We can land this safely without verifying that, but should check before we start using it anywhere.

Test Plan: Ran unit tests.

Reviewers: btrahan, nh, jungejason

Reviewed By: btrahan

CC: aran, epriestley

Maniphest Tasks: T605

Differential Revision: https://secure.phabricator.com/D1515

Details

Committed
epriestley <git@epriestley.com>Jan 31 2012, 21:07
Pushed
aubortJan 31 2017, 17:16
Parents
rPH2c3a08b37b76: Owners -- kill tabs
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH4f018488ae79: Restore Lisk transactional methods (authored by epriestley <git@epriestley.com>).Jan 31 2012, 21:07