Task #2832
closed
- Assigned To changed from Brett Smith to Ward Vandewege
Reviewing 7b4d7fa
Would it be possible to tighten the try block like this?
try:
collection = arvados.CollectionReader(arvados.Keep.get(self.collection_locator))
except Exception as detail:
print("%s: error: %s" % (self.collection_locator,detail) )
else:
for s in collection.all_streams():
…
That would help make it clearer which part we actually expect to fail. It might be trickier if all_streams
could also fail, though.
That's my only comment, so whether you make that change or can't, I think you're good to merge.
Good point - I think I'd prefer to play it safe and wrap the whole thing in the try/except block. If an exception is triggered there, the fuse driver just dies. That should be avoided, imo. Thanks for the review!
- Status changed from New to Resolved
- Start date set to 05/15/2014
- Remaining (hours) set to 0.0
Also available in: Atom
PDF