Transactions api needs to be a bit more consistent

We sometimes accept a const reference for something, then copy that into the captures of a lambda internally.   Other times, we just take an object (so already a copy) and move it into the captures internally.   Lets just settle on one way (the latter seems consistent with rest of api).