@fasseg I was thinking that originally too, but benchtool currently instantiates a new FedoraRestClient with every ActionWorker created.  Perhaps we should look at refactoring it to share FedoraRestClients across workers?  I'd be willing to make that change if it seems reasonable to you.

Anyways, the reason I am assigning a tx to a rest client is to ensure that the worker and the client use the same transaction, but you're right that I could do this by passing the tx in as a parameter rather than setting it on the client.  This would need to be the case if we switch over to sharing clients.

@ajs6f Its true that currently each benchtool rest client is using one tx at a time, but in the current implementation there are many rest clients, so there can be any number of transactions going on at once.