Re: Review Request 18935: Part 1 of converting from Packages to Metadata.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/#review36737 --- Ship it! Ship It! - Bill Farner On March 11, 2014, 12:09 a.m., Maxim Khutornenko wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/ --- (Updated March 11, 2014, 12:09 a.m.) Review request for Aurora, Kevin Sweeney, Suman Karumuri, and Bill Farner. Bugs: AURORA-158 https://issues.apache.org/jira/browse/AURORA-158 Repository: aurora Description --- Part 1 of converting from Packages to Metadata. Dual read from packages and metadata during the migration period. Client side changes will be addressed in Part 2. Diffs - src/main/java/org/apache/aurora/scheduler/http/SchedulerzJob.java c14e19513f6b1cd16d40c81d6c027ef495a16ee9 src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 60b2259f21b598fa38bec5a590516cba2c07e1ac src/main/java/org/apache/aurora/scheduler/http/TransformationUtils.java b6ee04ed8dc98f0fe47fabfd607d884ee1053bc5 src/main/java/org/apache/aurora/scheduler/local/IsolatedSchedulerModule.java fe2965d674c6282d936b3f1b02949aab35978134 src/main/resources/org/apache/aurora/scheduler/http/schedulerzrole.st d568a7e1bc836520bb21f28a329701ae37d59165 src/main/thrift/org/apache/aurora/gen/api.thrift d72b28c3378a651a8cff49216c1435ce7aee5977 src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java 4beb2cf2bcf8b902486b802ab0123541085a95e5 src/test/resources/org/apache/aurora/gen/api.thrift.md5 2308ba8da96197d41040ba772ea871003615698a Diff: https://reviews.apache.org/r/18935/diff/ Testing --- gradle build gradle run File Attachments Role https://reviews.apache.org/media/uploaded/files/2014/03/11/15f358ce-7b08-494f-a0c3-160a304cca91__Metadata_role.png Job https://reviews.apache.org/media/uploaded/files/2014/03/11/7ca09c15-4eca-4ceb-a6fe-fd31da3285fc__Metadata_job.png Thanks, Maxim Khutornenko
Review Request 18935: Part 1 of converting from Packages to Metadata.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/ --- Review request for Aurora, Suman Karumuri and Bill Farner. Bugs: AURORA-158 https://issues.apache.org/jira/browse/AURORA-158 Repository: aurora Description --- Part 1 of converting from Packages to Metadata. Dual read from packages and metadata during the migration period. Client side changes will be addressed in Part 2. Diffs - src/main/java/org/apache/aurora/scheduler/http/SchedulerzJob.java c14e19513f6b1cd16d40c81d6c027ef495a16ee9 src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 60b2259f21b598fa38bec5a590516cba2c07e1ac src/main/java/org/apache/aurora/scheduler/http/TransformationUtils.java b6ee04ed8dc98f0fe47fabfd607d884ee1053bc5 src/main/java/org/apache/aurora/scheduler/local/IsolatedSchedulerModule.java fe2965d674c6282d936b3f1b02949aab35978134 src/main/resources/org/apache/aurora/scheduler/http/schedulerzrole.st d568a7e1bc836520bb21f28a329701ae37d59165 src/main/thrift/org/apache/aurora/gen/api.thrift d72b28c3378a651a8cff49216c1435ce7aee5977 src/test/resources/org/apache/aurora/gen/api.thrift.md5 2308ba8da96197d41040ba772ea871003615698a Diff: https://reviews.apache.org/r/18935/diff/ Testing --- gradle build gradle run File Attachments Role https://reviews.apache.org/media/uploaded/files/2014/03/08/790daac1-89ac-436f-8c92-5e5dc7483154__Metadata_Role.png Job https://reviews.apache.org/media/uploaded/files/2014/03/08/7ca45940-666f-439b-867f-b89f42b11a38__Metadata_Job.png Thanks, Maxim Khutornenko
Re: Review Request 18935: Part 1 of converting from Packages to Metadata.
On March 8, 2014, 12:56 a.m., Bill Farner wrote: src/main/thrift/org/apache/aurora/gen/api.thrift, line 179 https://reviews.apache.org/r/18935/diff/1/?file=513979#file513979line179 feel free to rename this to packagesDEPRECATED That would require client changes that I would rather avoid in this changelist. On March 8, 2014, 12:56 a.m., Bill Farner wrote: src/main/thrift/org/apache/aurora/gen/api.thrift, line 182 https://reviews.apache.org/r/18935/diff/1/?file=513979#file513979line182 Does a map make sense instead? Thought about that but decided in favor of set as a more extensible approach (i.e. in case we decide to add a timestamp or any other attribute). - Maxim --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/#review36589 --- On March 8, 2014, 12:28 a.m., Maxim Khutornenko wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/ --- (Updated March 8, 2014, 12:28 a.m.) Review request for Aurora, Suman Karumuri and Bill Farner. Bugs: AURORA-158 https://issues.apache.org/jira/browse/AURORA-158 Repository: aurora Description --- Part 1 of converting from Packages to Metadata. Dual read from packages and metadata during the migration period. Client side changes will be addressed in Part 2. Diffs - src/main/java/org/apache/aurora/scheduler/http/SchedulerzJob.java c14e19513f6b1cd16d40c81d6c027ef495a16ee9 src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 60b2259f21b598fa38bec5a590516cba2c07e1ac src/main/java/org/apache/aurora/scheduler/http/TransformationUtils.java b6ee04ed8dc98f0fe47fabfd607d884ee1053bc5 src/main/java/org/apache/aurora/scheduler/local/IsolatedSchedulerModule.java fe2965d674c6282d936b3f1b02949aab35978134 src/main/resources/org/apache/aurora/scheduler/http/schedulerzrole.st d568a7e1bc836520bb21f28a329701ae37d59165 src/main/thrift/org/apache/aurora/gen/api.thrift d72b28c3378a651a8cff49216c1435ce7aee5977 src/test/resources/org/apache/aurora/gen/api.thrift.md5 2308ba8da96197d41040ba772ea871003615698a Diff: https://reviews.apache.org/r/18935/diff/ Testing --- gradle build gradle run File Attachments Role https://reviews.apache.org/media/uploaded/files/2014/03/08/790daac1-89ac-436f-8c92-5e5dc7483154__Metadata_Role.png Job https://reviews.apache.org/media/uploaded/files/2014/03/08/7ca45940-666f-439b-867f-b89f42b11a38__Metadata_Job.png Thanks, Maxim Khutornenko
Re: Review Request 18935: Part 1 of converting from Packages to Metadata.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/ --- (Updated March 8, 2014, 2:10 a.m.) Review request for Aurora, Kevin Sweeney, Suman Karumuri, and Bill Farner. Changes --- + Kevin. Bugs: AURORA-158 https://issues.apache.org/jira/browse/AURORA-158 Repository: aurora Description --- Part 1 of converting from Packages to Metadata. Dual read from packages and metadata during the migration period. Client side changes will be addressed in Part 2. Diffs (updated) - src/main/java/org/apache/aurora/scheduler/http/SchedulerzJob.java c14e19513f6b1cd16d40c81d6c027ef495a16ee9 src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 60b2259f21b598fa38bec5a590516cba2c07e1ac src/main/java/org/apache/aurora/scheduler/http/TransformationUtils.java b6ee04ed8dc98f0fe47fabfd607d884ee1053bc5 src/main/java/org/apache/aurora/scheduler/local/IsolatedSchedulerModule.java fe2965d674c6282d936b3f1b02949aab35978134 src/main/resources/org/apache/aurora/scheduler/http/schedulerzrole.st d568a7e1bc836520bb21f28a329701ae37d59165 src/main/thrift/org/apache/aurora/gen/api.thrift d72b28c3378a651a8cff49216c1435ce7aee5977 src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java 4beb2cf2bcf8b902486b802ab0123541085a95e5 src/test/resources/org/apache/aurora/gen/api.thrift.md5 2308ba8da96197d41040ba772ea871003615698a Diff: https://reviews.apache.org/r/18935/diff/ Testing --- gradle build gradle run File Attachments Role https://reviews.apache.org/media/uploaded/files/2014/03/08/790daac1-89ac-436f-8c92-5e5dc7483154__Metadata_Role.png Job https://reviews.apache.org/media/uploaded/files/2014/03/08/7ca45940-666f-439b-867f-b89f42b11a38__Metadata_Job.png Thanks, Maxim Khutornenko
Re: Review Request 18935: Part 1 of converting from Packages to Metadata.
On March 8, 2014, 12:56 a.m., Bill Farner wrote: src/main/thrift/org/apache/aurora/gen/api.thrift, line 179 https://reviews.apache.org/r/18935/diff/1/?file=513979#file513979line179 feel free to rename this to packagesDEPRECATED Maxim Khutornenko wrote: That would require client changes that I would rather avoid in this changelist. Keep forgetting thrift does not care about that. Changed. On March 8, 2014, 12:56 a.m., Bill Farner wrote: src/main/thrift/org/apache/aurora/gen/api.thrift, line 182 https://reviews.apache.org/r/18935/diff/1/?file=513979#file513979line182 Does a map make sense instead? Maxim Khutornenko wrote: Thought about that but decided in favor of set as a more extensible approach (i.e. in case we decide to add a timestamp or any other attribute). I am fine either way. Kevin, what's your take on it? - Maxim --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/#review36589 --- On March 8, 2014, 2:10 a.m., Maxim Khutornenko wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/ --- (Updated March 8, 2014, 2:10 a.m.) Review request for Aurora, Kevin Sweeney, Suman Karumuri, and Bill Farner. Bugs: AURORA-158 https://issues.apache.org/jira/browse/AURORA-158 Repository: aurora Description --- Part 1 of converting from Packages to Metadata. Dual read from packages and metadata during the migration period. Client side changes will be addressed in Part 2. Diffs - src/main/java/org/apache/aurora/scheduler/http/SchedulerzJob.java c14e19513f6b1cd16d40c81d6c027ef495a16ee9 src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 60b2259f21b598fa38bec5a590516cba2c07e1ac src/main/java/org/apache/aurora/scheduler/http/TransformationUtils.java b6ee04ed8dc98f0fe47fabfd607d884ee1053bc5 src/main/java/org/apache/aurora/scheduler/local/IsolatedSchedulerModule.java fe2965d674c6282d936b3f1b02949aab35978134 src/main/resources/org/apache/aurora/scheduler/http/schedulerzrole.st d568a7e1bc836520bb21f28a329701ae37d59165 src/main/thrift/org/apache/aurora/gen/api.thrift d72b28c3378a651a8cff49216c1435ce7aee5977 src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java 4beb2cf2bcf8b902486b802ab0123541085a95e5 src/test/resources/org/apache/aurora/gen/api.thrift.md5 2308ba8da96197d41040ba772ea871003615698a Diff: https://reviews.apache.org/r/18935/diff/ Testing --- gradle build gradle run File Attachments Role https://reviews.apache.org/media/uploaded/files/2014/03/08/790daac1-89ac-436f-8c92-5e5dc7483154__Metadata_Role.png Job https://reviews.apache.org/media/uploaded/files/2014/03/08/7ca45940-666f-439b-867f-b89f42b11a38__Metadata_Job.png Thanks, Maxim Khutornenko