why did incrementing the failed metrics move to the auction package?  It's weird that the auction package now imports the auctioneer in order to submit failure metrics which are really the domain of the auctioneer... why not keep it in `DistributedBatch`?