Re: Review Request 64625: Add a storage recovery tool

2017-12-14 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64625/#review193879 --- Ship it! Excellent addition. High-level approach LGTM. -

Re: Review Request 64629: Use java.util.Optional throughout

2017-12-14 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64629/#review193861 --- Master (5f79f7c) is green with this patch.

Review Request 64629: Use java.util.Optional throughout

2017-12-14 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64629/ --- Review request for Aurora and Jordan Ly. Repository: aurora Description

Re: Review Request 64625: Add a storage recovery tool

2017-12-14 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64625/#review193853 --- Ship it! Master (5f79f7c) is green with this patch.

Review Request 64625: Add a storage recovery tool

2017-12-14 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64625/ --- Review request for Aurora, Jordan Ly and Santhosh Kumar Shanmugham.

Re: Review Request 64519: Add a test to detect incompatible storage changes

2017-12-14 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64519/#review193802 --- I am traveling for the next 5 days, so my review will be delayed.

Re: Review Request 64341: Add metadata field to Job object in DSL

2017-12-14 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64341/#review193801 --- I am traveling for the next 5 days, so my review will be delayed.