Re: Review Request 64283: Remove redundant transaction recorder

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

Review Request 64283: Remove redundant transaction recorder

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

Re: Review Request 64234: Extract a storage Persistence layer

2017-12-02 Thread Jordan Ly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64234/#review192638 --- Ship it! Ship It! - Jordan Ly On Dec. 1, 2017, 12:32 a.m.,

Re: Review Request 64234: Extract a storage Persistence layer

2017-12-02 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64234/#review192636 --- Ship it! Ship It! - Stephan Erb On Dez. 1, 2017, 1:32

Re: Review Request 64234: Extract a storage Persistence layer

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

Re: Review Request 64234: Extract a storage Persistence layer

2017-12-02 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64234/#review192634 --- @ReviewBot retry - Bill Farner On Nov. 30, 2017, 4:32 p.m.,

Re: Review Request 64234: Extract a storage Persistence layer

2017-12-02 Thread Bill Farner
> On Dec. 1, 2017, 1:05 a.m., Stephan Erb wrote: > > This looks like a good improvement to me. Thanks! > > > > One general note: We have to ensure that > > https://github.com/apache/aurora/blob/master/docs/operations/storage.md > > will be up to date eventually. This does not need to happen