Darrell Bishop
Darrell Bishop
looks much better to me! On Fri, Dec 27, 2019 at 2:09 PM Tim Burke wrote: > *@tipabu* commented on this pull request. > > OK, I've thought about it...
I think this is a duplicate of Issue #46
@klauspost , I left the flag non-hidden as even though this one mentions GCS, but the other use-case was at the same tier-1 CSP where we hit the uint16 thread...
That's fair. I think the use-cases are valid, at least for using Warp for large-scale testing with CSPs. For the code volume & complexity, that naturally arose from maintaining seamless...
@klauspost, Hi, I did find an additional improvement to be had in refactoring the logic for list-existing between bench.{delete,get,stat} and did so with a helper function and options struct. I...
@klauspost Added one more tweak to clean up the separation of concerns between generator.{generator,random}. Now all the prefix-calculation is in generator.random, not spread across the two files. This will be...
@klauspost Hi, any thoughts on this patch after the 2 adjustments last week? Thanks, Darrell
@klauspost @harshavardhana Hedging my bets on this PR, I created #434 to just cover the --list-existing refactoring. I think that's an objective improvement in maintainability, even if not considering this...