Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33279/#review81525 --- Ship it! Ship It! - Maxim Khutornenko On April 16, 2015, 10 p.m.

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

> On April 24, 2015, 6:35 p.m., Bill Farner wrote: > > Ping - Maxim, can you check my replies above? > > Maxim Khutornenko wrote: > >My plan is to avoid changing behavior with the task store, so i did not > have any intention of removing fields. > > Sorry, I was under assumption yo

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

> On April 24, 2015, 6:35 p.m., Bill Farner wrote: > > Ping - Maxim, can you check my replies above? > > Maxim Khutornenko wrote: > >My plan is to avoid changing behavior with the task store, so i did not > have any intention of removing fields. > > Sorry, I was under assumption yo

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

> On April 24, 2015, 6:35 p.m., Bill Farner wrote: > > Ping - Maxim, can you check my replies above? >My plan is to avoid changing behavior with the task store, so i did not have >any intention of removing fields. Sorry, I was under assumption your decided to address the "user" part. Not stor

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33279/#review81512 --- Ping - Maxim, can you check my replies above? - Bill Farner On Ap

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33279/#review80750 --- src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sq

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

> On April 20, 2015, 5:50 p.m., Maxim Khutornenko wrote: > > src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql, line > > 200 > > > > > > Isn't this too restrictive? Currently, we can still have some

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

> On April 20, 2015, 5:50 p.m., Maxim Khutornenko wrote: > > src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql, line > > 179 > > > > > > What are your plans on retaining the "user" part of the Ident

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33279/#review80409 --- Ship it! Ship It! - Kevin Sweeney On April 16, 2015, 3 p.m., Bil

Re: Review Request 33279: Add SQL tables needed for a datbase-backed task store.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33279/#review80387 --- Master (b18dc44) is green with this patch. ./build-support/jenkins

Review Request 33279: Add SQL tables needed for a datbase-backed task store.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33279/ --- Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Bugs: AURORA-55