[PATCH v3 00/19] Use ref transactions from most callers · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 02/19] refs.c: allow passing NULL to ref_transaction_free · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 11/19] tag.c: use ref transactions when doing updates · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 11/19] tag.c: use ref transactions when doing updates · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v3 11/19] tag.c: use ref transactions when doing updates · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 07/19] refs.c: remove the onerr argument to ref_transaction_commit · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 07/19] refs.c: remove the onerr argument to ref_transaction_commit · Jonathan Nieder <hidden> · 2016-06-15
Re: [PATCH v3 07/19] refs.c: remove the onerr argument to ref_transaction_commit · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 12/19] replace.c: use the ref transaction functions for updates · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 12/19] replace.c: use the ref transaction functions for updates · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v3 12/19] replace.c: use the ref transaction functions for updates · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 19/19] refs.c: pass **transaction to commit and have it clear the pointer · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 19/19] refs.c: pass **transaction to commit and have it clear the pointer · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v3 19/19] refs.c: pass **transaction to commit and have it clear the pointer · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 19/19] refs.c: pass **transaction to commit and have it clear the pointer · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v3 19/19] refs.c: pass **transaction to commit and have it clear the pointer · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 19/19] refs.c: pass **transaction to commit and have it clear the pointer · Michael Haggerty <hidden> · 2016-06-15
[PATCH v3 08/19] refs.c: change ref_transaction_update() to do error checking and return status · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 17/19] refs.c: change update_ref to use a transaction · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 17/19] refs.c: change update_ref to use a transaction · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v3 17/19] refs.c: change update_ref to use a transaction · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 09/19] refs.c: change ref_transaction_create to do error checking and return status · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 18/19] refs.c: free the transaction before returning when number of updates is 0 · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 05/19] update-ref.c: use the error string from _commit to print better message · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 05/19] update-ref.c: use the error string from _commit to print better message · Jonathan Nieder <hidden> · 2016-06-15
Re: [PATCH v3 05/19] update-ref.c: use the error string from _commit to print better message · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 03/19] refs.c: make ref_transaction_commit return an error string · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 03/19] refs.c: make ref_transaction_commit return an error string · Jonathan Nieder <hidden> · 2016-06-15
Re: [PATCH v3 03/19] refs.c: make ref_transaction_commit return an error string · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 04/19] refs.c: return error string from ref_update_reject_duplicates on failure · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 04/19] refs.c: return error string from ref_update_reject_duplicates on failure · Jonathan Nieder <hidden> · 2016-06-15
Re: [PATCH v3 04/19] refs.c: return error string from ref_update_reject_duplicates on failure · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 10/19] refs.c: ref_transaction_delete to check for error and return status · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 13/19] commit.c: use ref transactions for updates · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 14/19] sequencer.c: use ref transactions for all ref updates · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 06/19] refs.c: make update_ref_write to return error string on failure · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 06/19] refs.c: make update_ref_write to return error string on failure · Jonathan Nieder <hidden> · 2016-06-15
Re: [PATCH v3 06/19] refs.c: make update_ref_write to return error string on failure · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 15/19] fast-import.c: change update_branch to use ref transactions · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 01/19] refs.c: constify the sha arguments for ref_transaction_create|delete|update · Ronnie Sahlberg <hidden> · 2016-06-15
[PATCH v3 16/19] branch.c: use ref transaction for all ref updates · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 16/19] branch.c: use ref transaction for all ref updates · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v3 16/19] branch.c: use ref transaction for all ref updates · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 16/19] branch.c: use ref transaction for all ref updates · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v3 16/19] branch.c: use ref transaction for all ref updates · Ronnie Sahlberg <hidden> · 2016-06-15
Re: [PATCH v3 00/19] Use ref transactions from most callers · Jonathan Nieder <hidden> · 2016-06-15
Re: [PATCH v3 00/19] Use ref transactions from most callers · Ronnie Sahlberg <hidden> · 2016-06-15

Re: [PATCH v3 19/19] refs.c: pass **transaction to commit and have it clear the pointer

From: Michael Haggerty <hidden>
Date: 2016-06-15 23:00:56

On 04/29/2014 08:58 PM, Ronnie Sahlberg wrote:

On Tue, Apr 29, 2014 at 2:25 AM, Michael Haggerty [off-list ref] wrote:

quoted

[...]
Unless we want to make ref_transaction objects reusable.  [...]

ACK, but I don't think we need reusable transaction yet. Once the API
is cleaned up
it should be reasonably easy to add in the future if we see a need for it.
Sounds reasonable to you ?

Yes, reusable transaction objects would be pretty straightforward to
retrofit, so I agree that there is no need to implement it now.

Michael

-- 
Michael Haggerty
mhagger@alum.mit.edu
http://softwareswirl.blogspot.com/

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help