Re: Review Request 64286: Recover snapshots via the Op stream

> On Dec. 14, 2017, 2:07 a.m., Jordan Ly wrote: > > src/main/java/org/apache/aurora/scheduler/storage/backup/TemporaryStorage.java > > Line 88 (original), 90 (patched) > > > > > > We shouldn't map Edit -> op > >

Re: Review Request 64286: Recover snapshots via the Op stream

> On Dec. 13, 2017, 6:07 p.m., Jordan Ly wrote: > > Small nits but overall LGTM. In addition to the end-to-end tests passing, > > could you also do a sanity check that this patch works fine on a small > > cluster (upgrade to this patch and downgrade to the previous version, > > ensuring

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/#review193756 --- Ship it! Master (4489dc3) is green with this patch.

Re: Review Request 64286: Recover snapshots via the Op stream

> On Dec. 13, 2017, 6:07 p.m., Jordan Ly wrote: > > Small nits but overall LGTM. In addition to the end-to-end tests passing, > > could you also do a sanity check that this patch works fine on a small > > cluster (upgrade to this patch and downgrade to the previous version, > > ensuring

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/#review193751 --- Fix it, then Ship it! Small nits but overall LGTM. In addition

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/#review193740 --- Jordan - any remaining comments on the patch? - Bill Farner On

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/#review193625 --- Ship it! As the patch is pretty large and mostly a refactoring

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/#review193577 --- Ship it! Master (301f066) is green with this patch.

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/ --- (Updated Dec. 12, 2017, 9:46 a.m.) Review request for Aurora, Jordan Ly and

Re: Review Request 64286: Recover snapshots via the Op stream

> On Dec. 8, 2017, 1:34 p.m., Jordan Ly wrote: > > src/main/java/org/apache/aurora/scheduler/storage/durability/DurableStorage.java > > Line 258 (original), 154 (patched) > > > > > > Why do we grab the writeLock

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/#review193211 ---

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/#review192892 --- Ship it! Master (a0628ef) is green with this patch.

Re: Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/#review192888 --- Reviewer notes.

Review Request 64286: Recover snapshots via the Op stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64286/ --- Review request for Aurora, Jordan Ly and Stephan Erb. Repository: aurora