Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder

2018-03-18 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65769/#review199394 --- Ship it! Ship It! - Stephan Erb On Feb. 23, 2018, 7:04

Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder

2018-03-18 Thread Stephan Erb
> On March 16, 2018, 7:02 p.m., Stephan Erb wrote: > > Is this RecoveryTool just a prototype or used for real? If it is the > > preferred backup way now, we have to update > > https://github.com/apache/aurora/blob/master/docs/operations/backup-restore.md. > > Jordan Ly wrote: > I would

Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder

2018-03-17 Thread Jordan Ly
> On March 16, 2018, 6:02 p.m., Stephan Erb wrote: > > Is this RecoveryTool just a prototype or used for real? If it is the > > preferred backup way now, we have to update > > https://github.com/apache/aurora/blob/master/docs/operations/backup-restore.md. I would call it a prototype for now.

Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder

2018-03-16 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65769/#review199333 --- Is this RecoveryTool just a prototype or used for real? If it is

Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder

2018-03-14 Thread Renan DelValle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65769/#review199222 --- Ship it! Ship It! - Renan DelValle On Feb. 23, 2018, 10:04

Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder

2018-03-14 Thread Jordan Ly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65769/#review199219 --- bump - Jordan Ly On Feb. 23, 2018, 6:04 p.m., Jordan Ly wrote: