Re: Review Request 63743: Remove the internal SQL database

2017-11-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63743/#review190883 --- Master (e0624b2) is green with this patch.

Re: Review Request 63743: Remove the internal SQL database

2017-11-13 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63743/#review190879 --- Ship it! Ship It! - David McLaughlin On Nov. 11, 2017, 4:53

Re: Review Request 63743: Remove the internal SQL database

2017-11-13 Thread Jordan Ly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63743/#review190845 --- Ship it! Ship It! - Jordan Ly On Nov. 11, 2017, 4:53 a.m.,

Re: Review Request 63743: Remove the internal SQL database

2017-11-12 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63743/#review190789 --- Master (9b9b2ee) is green with this patch.

Re: Review Request 63743: Remove the internal SQL database

2017-11-11 Thread Bill Farner
> On Nov. 11, 2017, 11:14 a.m., Jordan Ly wrote: > > Should we remove `GatingDelayExecutor`/`GatedWorkQueue` as well? I believe > > they are only used in DB stores. Sure, done. This required a bit of a refactor, so you'll notice the diff has grown a bit more complex. - Bill

Re: Review Request 63743: Remove the internal SQL database

2017-11-11 Thread Jordan Ly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63743/#review190785 --- Should we remove `GatingDelayExecutor`/`GatedWorkQueue` as well?

Re: Review Request 63743: Remove the internal SQL database

2017-11-11 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63743/#review190779 --- Master (9b9b2ee) is green with this patch.

Re: Review Request 63743: Remove the internal SQL database

2017-11-11 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63743/#review190778 --- Ship it! Ship It! - Stephan Erb On Nov. 11, 2017, 5:53

Review Request 63743: Remove the internal SQL database

2017-11-10 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63743/ --- Review request for Aurora, David McLaughlin, Jordan Ly, and Stephan Erb.