LGTM modulo a single suggestion.
https://codereview.appspot.com/10550043/diff/1/worker/resumer/resumer_test.go File worker/resumer/resumer_test.go (right):
https://codereview.appspot.com/10550043/diff/1/worker/resumer/resumer_test.go#newcode47 worker/resumer/resumer_test.go:47: c.Assert(len(tr.timestamps) > 0, Equals, true) This code block is a bit obscure to my puny mind - can you put some comment describing what we're actually doing here please?
https://codereview.appspot.com/10550043/
« Back to merge proposal
LGTM modulo a single suggestion.
https:/ /codereview. appspot. com/10550043/ diff/1/ worker/ resumer/ resumer_ test.go resumer/ resumer_ test.go (right):
File worker/
https:/ /codereview. appspot. com/10550043/ diff/1/ worker/ resumer/ resumer_ test.go# newcode47 resumer/ resumer_ test.go: 47: c.Assert( len(tr. timestamps) > 0,
worker/
Equals, true)
This code block is a bit obscure to my puny mind - can you put some
comment describing what we're actually doing here please?
https:/ /codereview. appspot. com/10550043/