> On Aug. 4, 2014, 9:41 p.m., Bill Farner wrote: > > src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql, line > > 98 > > <https://reviews.apache.org/r/24243/diff/4/?file=650982#file650982line98> > > > > Strongly consider dropping update_id to let the IDENTITY stand alone.
If we were not returning the ID within the same call I would definitely go for the IDENTITY as ID. However, after spending hours trying to make MERGE statement work correctly with the auto-generated ID I had to give up. Also, the updateID appears consistent with taskID where the value is generated before accessing the store. > On Aug. 4, 2014, 9:41 p.m., Bill Farner wrote: > > src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql, line > > 101 > > <https://reviews.apache.org/r/24243/diff/4/?file=650982#file650982line101> > > > > Feel free to pull the comment up to the top, to signify that we want to > > change all to TIMESTAMP. Done. > On Aug. 4, 2014, 9:41 p.m., Bill Farner wrote: > > src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql, line > > 117 > > <https://reviews.apache.org/r/24243/diff/4/?file=650982#file650982line117> > > > > Feel free to punt in this review, but BLOB might be preferable here. Thanks for the idea. Will definitely consider when dealing with TaskConfig serialization. > On Aug. 4, 2014, 9:41 p.m., Bill Farner wrote: > > src/main/thrift/org/apache/aurora/gen/api.thrift, line 562 > > <https://reviews.apache.org/r/24243/diff/4/?file=650983#file650983line562> > > > > s/True/true/, which is the keyword thrift uses. Done. - Maxim ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24243/#review49510 ----------------------------------------------------------- On Aug. 4, 2014, 9:04 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24243/ > ----------------------------------------------------------- > > (Updated Aug. 4, 2014, 9:04 p.m.) > > > Review request for Aurora, David McLaughlin, Kevin Sweeney, and Bill Farner. > > > Bugs: AURORA-612 > https://issues.apache.org/jira/browse/AURORA-612 > > > Repository: aurora > > > Description > ------- > > DB tables for the job update store. Sending out early to solicit feedback > before moving to mappers. > > > Diffs > ----- > > src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql > 5358b45102f53eea97a1ca709ba9375daa91a3ef > src/main/thrift/org/apache/aurora/gen/api.thrift > 595c8dfc3d28c2bfa92ae2226a28a8519d75b8c4 > src/test/python/apache/aurora/client/api/test_scheduler_client.py > e1f1a95d1757d584056fe6be9c98049444f6a75b > > Diff: https://reviews.apache.org/r/24243/diff/ > > > Testing > ------- > > > Thanks, > > Maxim Khutornenko > >
