-----------------------------------------------------------
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.sql
<https://reviews.apache.org/r/33279/#comment130862>

    What are your plans on retaining the "user" part of the Identity struct 
(AURORA-749)? If you are planning to address it later a follow up TODO here 
would be nice to avoid dropping this issue off the radar.



src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql
<https://reviews.apache.org/r/33279/#comment130882>

    Isn't this too restrictive? Currently, we can still have something like 
'rack':'limit:5' and 'rack':'abc'. This constraint will now allow configs like 
that any longer.


- Maxim Khutornenko


On April 16, 2015, 10 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33279/
> -----------------------------------------------------------
> 
> (Updated April 16, 2015, 10 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Maxim Khutornenko.
> 
> 
> Bugs: AURORA-556
>     https://issues.apache.org/jira/browse/AURORA-556
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Add SQL tables needed for a datbase-backed task store.
> 
> 
> Diffs
> -----
> 
>   src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql 
> ed73ef7a25537f145910dba7a1985805b3979173 
> 
> Diff: https://reviews.apache.org/r/33279/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Reply via email to