Andrew Bennetts wrote: [...] > At a glance it wouldn't be too outrageously messy to make those changes > to lib/lp/codehosting/vfs/transport.py, so I'll give it a go and propose > a patch to lp too. Aside from having traceback-less Failures it should > have no change to semantics.
You can find a patch (untested, sorry!) that does this in <lp:~spiv/launchpad/codehosting-cheaper-failures>. I don't think it'll avoid the expensive cleanFailures entirely, but hopefully will get 90% of them. It's not too large: codehosting/vfs/branchfs.py | 33 +++++++++++++++++++++------------ codehosting/vfs/branchfsclient.py | 3 ++- codehosting/vfs/transport.py | 20 ++++++++++++++++---- services/twistedsupport/__init__.py | 25 ++++++++++++++++++++++--- 4 files changed, 61 insertions(+), 20 deletions(-) John, could you give this a try and see if it helps? -Andrew. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-dev More help : https://help.launchpad.net/ListHelp
