Review Request 26394: Deprecating Identity struct (renaming fields).

2014-10-06 Thread Maxim Khutornenko

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26394/
---

Review request for Aurora, David McLaughlin, Mark Chu-Carroll, and Bill Farner.


Bugs: AURORA-84
https://issues.apache.org/jira/browse/AURORA-84


Repository: aurora


Description
---

Part 1 of Identity struct deprecation: renaming fields. Will require some 
rebasing when https://reviews.apache.org/r/26308 lands.


Diffs
-

  examples/jobs/hello_world.aurora fc7877c3a60e56e301d9ee1fabd73446afca7236 
  src/main/java/org/apache/aurora/scheduler/TaskIdGenerator.java 
5c75cc8cae53edfa069c85c37ebad34774682081 
  src/main/java/org/apache/aurora/scheduler/async/Preemptor.java 
e9f251508257cd7287ff00773e0073a3cd130df8 
  src/main/java/org/apache/aurora/scheduler/base/JobKeys.java 
a76c3fac71b35115064fba6644cff0066fd9e630 
  src/main/java/org/apache/aurora/scheduler/base/Query.java 
eded7a59eb394748b93d7fbc085a1bdf64b043cc 
  
src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
 865742171c11fbe5cf1469a69dd7258ec1be28c2 
  src/main/java/org/apache/aurora/scheduler/http/Utilization.java 
a0cb7bf56aeb7edd92b25d8d69a739d87452777a 
  src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java 
8c20ab6f2bebf1d1c0f91fed3f1e48361cdf45d6 
  src/main/java/org/apache/aurora/scheduler/storage/mem/MemTaskStore.java 
37176237fac336413267f3c8bb4e1b9a6255150c 
  
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 
5dcae4a6132026504cf02093ad4c68ab521e4ab8 
  src/main/python/apache/aurora/client/api/__init__.py 
26300792594e4005dacc139a9f89711b8a66ab61 
  src/main/python/apache/aurora/client/api/command_runner.py 
a1fed5fc75dde3a79c840515e6daa4741156ef97 
  src/main/python/apache/aurora/client/api/instance_watcher.py 
b390aa8993205f1f6938f8c295e3c16a0bf4df6d 
  src/main/python/apache/aurora/client/api/job_monitor.py 
18d5c0381d43fc7b24bae4b2e5e6fdc774a74b52 
  src/main/python/apache/aurora/client/api/sla.py 
b9b64680b15f5395ed6aca681b9b1c30ffe2822c 
  src/main/python/apache/aurora/client/api/updater.py 
bf608981c2f2e7960b68c3fbda144277a59a3d40 
  src/main/python/apache/aurora/client/cli/task.py 
c41484bdc27266443bc4e139e1ebb362a59be0f9 
  src/main/python/apache/aurora/client/commands/admin.py 
deee0250f3ba9837feeb92acc654f5b3b68b4e0f 
  src/main/python/apache/aurora/client/commands/core.py 
58f419e674f1a9a0ae9da6faa2e39c8167bab597 
  src/main/python/apache/aurora/client/commands/ssh.py 
9ee94a0bdfd8bbc030ae978e2ac3fe39571b4ce2 
  src/main/python/apache/aurora/common/aurora_job_key.py 
a7ca7b6df6b9566b3ce617283ccac948deb2eb83 
  src/main/python/apache/aurora/config/thrift.py 
288fb40f65629c8fd4eb7d92c8bf02369237de3b 
  src/main/python/apache/aurora/executor/aurora_executor.py 
79a24855b2a68271b7478395dfdadab8755c3af2 
  src/main/python/apache/aurora/executor/common/announcer.py 
c466da8d48bbc2aa227c2db157cab84665ad6602 
  src/main/python/apache/aurora/executor/thermos_task_runner.py 
7b346e253677ee9b42c57782f7f67ff63b6a0083 
  src/main/resources/org/apache/aurora/scheduler/http/ui/js/controllers.js 
7b6018814798494acd4687fcd3dce32a9bf63308 
  src/main/resources/org/apache/aurora/scheduler/http/ui/js/services.js 
b744f375411e09b7f776e4a05ee5961227143439 
  src/main/thrift/org/apache/aurora/gen/api.thrift 
8794731f4b3f1033588bdfa33c292e4796319a2a 
  src/test/java/org/apache/aurora/scheduler/MesosTaskFactoryImplTest.java 
e96974764844b5d1a3a05f6996075fccee209594 
  src/test/java/org/apache/aurora/scheduler/TaskVarsTest.java 
371ae87f5954fa5f092db1f6d21e2291d7576173 
  src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
606c4434b7158220ccf1403b6deac939021fee31 
  src/test/java/org/apache/aurora/scheduler/async/GcExecutorLauncherTest.java 
f2d153f446247032ad9d8d173fb70870dbfdcca1 
  src/test/java/org/apache/aurora/scheduler/async/PreemptorImplTest.java 
8ee84cda8670d117e2efa2d1a114da6f0d8315d6 
  
src/test/java/org/apache/aurora/scheduler/async/RescheduleCalculatorImplTest.java
 131bd826dfe47f40f3c27f29c095ed42953e316c 
  src/test/java/org/apache/aurora/scheduler/async/TaskGroupsTest.java 
efdde15939b2a851e38be53cceab395cc2cd82a1 
  src/test/java/org/apache/aurora/scheduler/async/TaskHistoryPrunerTest.java 
53d2c6bb78ad08a84639c1ecd48ba64d17c3f9fc 
  src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerImplTest.java 
6534329a92bf005223fa8907cbe4a8a3a511e142 
  src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerTest.java 
9970948bace4c0ecbc51d6fc79270d77fb17bf87 
  src/test/java/org/apache/aurora/scheduler/base/TaskTestUtil.java 
b8b9c1b8cdf7b641976c583a71fdd9b0c14e6e5f 
  
src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java
 ee9587582bd7c45a446e8afe28930c18a97d2792 
  src/test/java/org/apache/aurora/scheduler/cron/quartz/CronIT.java 
f3c7c5bd53df759432beda4fa46db49fd0514b42 
  src

Re: Review Request 26394: Deprecating Identity struct (renaming fields).

2014-10-08 Thread Maxim Khutornenko

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26394/
---

(Updated Oct. 8, 2014, 11:39 p.m.)


Review request for Aurora, David McLaughlin, Mark Chu-Carroll, and Bill Farner.


Changes
---

Rebased.

Ping.


Bugs: AURORA-84
https://issues.apache.org/jira/browse/AURORA-84


Repository: aurora


Description (updated)
---

Part 1 of Identity struct deprecation: renaming fields.


Diffs (updated)
-

  examples/jobs/hello_world.aurora fc7877c3a60e56e301d9ee1fabd73446afca7236 
  src/main/java/org/apache/aurora/scheduler/TaskIdGenerator.java 
5c75cc8cae53edfa069c85c37ebad34774682081 
  src/main/java/org/apache/aurora/scheduler/async/Preemptor.java 
e9f251508257cd7287ff00773e0073a3cd130df8 
  src/main/java/org/apache/aurora/scheduler/base/JobKeys.java 
a76c3fac71b35115064fba6644cff0066fd9e630 
  src/main/java/org/apache/aurora/scheduler/base/Query.java 
eded7a59eb394748b93d7fbc085a1bdf64b043cc 
  
src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
 865742171c11fbe5cf1469a69dd7258ec1be28c2 
  src/main/java/org/apache/aurora/scheduler/http/Utilization.java 
a0cb7bf56aeb7edd92b25d8d69a739d87452777a 
  src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java 
8c20ab6f2bebf1d1c0f91fed3f1e48361cdf45d6 
  src/main/java/org/apache/aurora/scheduler/storage/mem/MemTaskStore.java 
37176237fac336413267f3c8bb4e1b9a6255150c 
  
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 
5dcae4a6132026504cf02093ad4c68ab521e4ab8 
  src/main/python/apache/aurora/client/api/instance_watcher.py 
b390aa8993205f1f6938f8c295e3c16a0bf4df6d 
  src/main/python/apache/aurora/client/api/sla.py 
b9b64680b15f5395ed6aca681b9b1c30ffe2822c 
  src/main/python/apache/aurora/client/cli/task.py 
c41484bdc27266443bc4e139e1ebb362a59be0f9 
  src/main/python/apache/aurora/client/commands/admin.py 
deee0250f3ba9837feeb92acc654f5b3b68b4e0f 
  src/main/python/apache/aurora/client/commands/core.py 
58f419e674f1a9a0ae9da6faa2e39c8167bab597 
  src/main/python/apache/aurora/client/commands/ssh.py 
d2b8bf675556b924d3d63b545d036dc48a081486 
  src/main/python/apache/aurora/config/thrift.py 
288fb40f65629c8fd4eb7d92c8bf02369237de3b 
  src/main/python/apache/aurora/executor/aurora_executor.py 
2c6423d096656f426a4385f4edef6875ebad7049 
  src/main/python/apache/aurora/executor/common/announcer.py 
74b2114d1ede7a4a4a68b78144f9b08bc994bd87 
  src/main/python/apache/aurora/executor/thermos_task_runner.py 
7b346e253677ee9b42c57782f7f67ff63b6a0083 
  src/main/resources/scheduler/assets/js/controllers.js 
7e9037ee921b009dc2b7c5adcf057bedebb01632 
  src/main/resources/scheduler/assets/js/services.js 
b744f375411e09b7f776e4a05ee5961227143439 
  src/main/thrift/org/apache/aurora/gen/api.thrift 
8794731f4b3f1033588bdfa33c292e4796319a2a 
  src/test/java/org/apache/aurora/scheduler/MesosTaskFactoryImplTest.java 
e96974764844b5d1a3a05f6996075fccee209594 
  src/test/java/org/apache/aurora/scheduler/TaskVarsTest.java 
371ae87f5954fa5f092db1f6d21e2291d7576173 
  src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
606c4434b7158220ccf1403b6deac939021fee31 
  src/test/java/org/apache/aurora/scheduler/async/GcExecutorLauncherTest.java 
f2d153f446247032ad9d8d173fb70870dbfdcca1 
  src/test/java/org/apache/aurora/scheduler/async/PreemptorImplTest.java 
8ee84cda8670d117e2efa2d1a114da6f0d8315d6 
  
src/test/java/org/apache/aurora/scheduler/async/RescheduleCalculatorImplTest.java
 131bd826dfe47f40f3c27f29c095ed42953e316c 
  src/test/java/org/apache/aurora/scheduler/async/TaskGroupsTest.java 
efdde15939b2a851e38be53cceab395cc2cd82a1 
  src/test/java/org/apache/aurora/scheduler/async/TaskHistoryPrunerTest.java 
53d2c6bb78ad08a84639c1ecd48ba64d17c3f9fc 
  src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerImplTest.java 
6534329a92bf005223fa8907cbe4a8a3a511e142 
  src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerTest.java 
9970948bace4c0ecbc51d6fc79270d77fb17bf87 
  src/test/java/org/apache/aurora/scheduler/base/TaskTestUtil.java 
b8b9c1b8cdf7b641976c583a71fdd9b0c14e6e5f 
  
src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java
 ee9587582bd7c45a446e8afe28930c18a97d2792 
  src/test/java/org/apache/aurora/scheduler/cron/quartz/CronIT.java 
f3c7c5bd53df759432beda4fa46db49fd0514b42 
  src/test/java/org/apache/aurora/scheduler/cron/quartz/QuartzTestUtil.java 
d2d3e86bb5acf3402f55188b9ae440412ef14b5a 
  
src/test/java/org/apache/aurora/scheduler/filter/SchedulingFilterImplTest.java 
6a9c4ee278ed3ee8222404504e571f20991c2ae2 
  src/test/java/org/apache/aurora/scheduler/http/api/ApiBetaTest.java 
62154045f49c5b23949dc739d735c3e5d3680b89 
  src/test/java/org/apache/aurora/scheduler/quota/QuotaManagerImplTest.java 
b58c8f363e6e7c72accaf590b2a7cb7bf24275ea 
  src/test/java/org/apache/aurora/scheduler/sla/Sl

Re: Review Request 26394: Deprecating Identity struct (renaming fields).

2014-10-08 Thread Bill Farner

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26394/#review55907
---


Thanks again for leading this - i'm very happy to see momentum on removing 
`Identity`.

Stepping back - i wonder if we should re-evaluate the way we do field 
deprecations now that we've established a protocol with JIRA and releases.  
This might mean we don't need to do the `DEPRECATED` mangling.  What do you 
think?

- Bill Farner


On Oct. 8, 2014, 11:39 p.m., Maxim Khutornenko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26394/
> ---
> 
> (Updated Oct. 8, 2014, 11:39 p.m.)
> 
> 
> Review request for Aurora, David McLaughlin, Mark Chu-Carroll, and Bill 
> Farner.
> 
> 
> Bugs: AURORA-84
> https://issues.apache.org/jira/browse/AURORA-84
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Part 1 of Identity struct deprecation: renaming fields.
> 
> 
> Diffs
> -
> 
>   examples/jobs/hello_world.aurora fc7877c3a60e56e301d9ee1fabd73446afca7236 
>   src/main/java/org/apache/aurora/scheduler/TaskIdGenerator.java 
> 5c75cc8cae53edfa069c85c37ebad34774682081 
>   src/main/java/org/apache/aurora/scheduler/async/Preemptor.java 
> e9f251508257cd7287ff00773e0073a3cd130df8 
>   src/main/java/org/apache/aurora/scheduler/base/JobKeys.java 
> a76c3fac71b35115064fba6644cff0066fd9e630 
>   src/main/java/org/apache/aurora/scheduler/base/Query.java 
> eded7a59eb394748b93d7fbc085a1bdf64b043cc 
>   
> src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
>  865742171c11fbe5cf1469a69dd7258ec1be28c2 
>   src/main/java/org/apache/aurora/scheduler/http/Utilization.java 
> a0cb7bf56aeb7edd92b25d8d69a739d87452777a 
>   src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java 
> 8c20ab6f2bebf1d1c0f91fed3f1e48361cdf45d6 
>   src/main/java/org/apache/aurora/scheduler/storage/mem/MemTaskStore.java 
> 37176237fac336413267f3c8bb4e1b9a6255150c 
>   
> src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
>  5dcae4a6132026504cf02093ad4c68ab521e4ab8 
>   src/main/python/apache/aurora/client/api/instance_watcher.py 
> b390aa8993205f1f6938f8c295e3c16a0bf4df6d 
>   src/main/python/apache/aurora/client/api/sla.py 
> b9b64680b15f5395ed6aca681b9b1c30ffe2822c 
>   src/main/python/apache/aurora/client/cli/task.py 
> c41484bdc27266443bc4e139e1ebb362a59be0f9 
>   src/main/python/apache/aurora/client/commands/admin.py 
> deee0250f3ba9837feeb92acc654f5b3b68b4e0f 
>   src/main/python/apache/aurora/client/commands/core.py 
> 58f419e674f1a9a0ae9da6faa2e39c8167bab597 
>   src/main/python/apache/aurora/client/commands/ssh.py 
> d2b8bf675556b924d3d63b545d036dc48a081486 
>   src/main/python/apache/aurora/config/thrift.py 
> 288fb40f65629c8fd4eb7d92c8bf02369237de3b 
>   src/main/python/apache/aurora/executor/aurora_executor.py 
> 2c6423d096656f426a4385f4edef6875ebad7049 
>   src/main/python/apache/aurora/executor/common/announcer.py 
> 74b2114d1ede7a4a4a68b78144f9b08bc994bd87 
>   src/main/python/apache/aurora/executor/thermos_task_runner.py 
> 7b346e253677ee9b42c57782f7f67ff63b6a0083 
>   src/main/resources/scheduler/assets/js/controllers.js 
> 7e9037ee921b009dc2b7c5adcf057bedebb01632 
>   src/main/resources/scheduler/assets/js/services.js 
> b744f375411e09b7f776e4a05ee5961227143439 
>   src/main/thrift/org/apache/aurora/gen/api.thrift 
> 8794731f4b3f1033588bdfa33c292e4796319a2a 
>   src/test/java/org/apache/aurora/scheduler/MesosTaskFactoryImplTest.java 
> e96974764844b5d1a3a05f6996075fccee209594 
>   src/test/java/org/apache/aurora/scheduler/TaskVarsTest.java 
> 371ae87f5954fa5f092db1f6d21e2291d7576173 
>   src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
> 606c4434b7158220ccf1403b6deac939021fee31 
>   src/test/java/org/apache/aurora/scheduler/async/GcExecutorLauncherTest.java 
> f2d153f446247032ad9d8d173fb70870dbfdcca1 
>   src/test/java/org/apache/aurora/scheduler/async/PreemptorImplTest.java 
> 8ee84cda8670d117e2efa2d1a114da6f0d8315d6 
>   
> src/test/java/org/apache/aurora/scheduler/async/RescheduleCalculatorImplTest.java
>  131bd826dfe47f40f3c27f29c095ed42953e316c 
>   src/test/java/org/apache/aurora/scheduler/async/TaskGroupsTest.java 
> efdde15939b2a851e38be53cceab395cc2cd82a1 
>   src/test/java/org/apache/aurora/scheduler/async/TaskHistoryPrunerTest.java 
> 53d2c6bb78ad08a84639c1ecd48ba64d17c3f9fc 
>   src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerImplTest.java 
> 6534329a92bf005223fa8907cbe4a8a3a511e142 
>   src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerTest.java 
> 9970948bace4c0ecbc51d6fc79270d77fb17bf87 
>   src/test/java/org/apache/aurora/scheduler/base/TaskTestUtil.java 
> b8b9c1b8cdf7b641976c583a71fdd9b0c14e6e5f 
> 

Re: Review Request 26394: Deprecating Identity struct (renaming fields).

2014-10-08 Thread Maxim Khutornenko


> On Oct. 9, 2014, 12:05 a.m., Bill Farner wrote:
> > Thanks again for leading this - i'm very happy to see momentum on removing 
> > `Identity`.
> > 
> > Stepping back - i wonder if we should re-evaluate the way we do field 
> > deprecations now that we've established a protocol with JIRA and releases.  
> > This might mean we don't need to do the `DEPRECATED` mangling.  What do you 
> > think?

I still think visual code reminder is quite beneficial in avoiding the use of 
deprecated fields. The between-release time is just too long to remember what 
needs to be avoided when coding far away from thrift schema. Besides, it helps 
validate the concept and will facilitate later removal.


- Maxim


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26394/#review55907
---


On Oct. 8, 2014, 11:39 p.m., Maxim Khutornenko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26394/
> ---
> 
> (Updated Oct. 8, 2014, 11:39 p.m.)
> 
> 
> Review request for Aurora, David McLaughlin, Mark Chu-Carroll, and Bill 
> Farner.
> 
> 
> Bugs: AURORA-84
> https://issues.apache.org/jira/browse/AURORA-84
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Part 1 of Identity struct deprecation: renaming fields.
> 
> 
> Diffs
> -
> 
>   examples/jobs/hello_world.aurora fc7877c3a60e56e301d9ee1fabd73446afca7236 
>   src/main/java/org/apache/aurora/scheduler/TaskIdGenerator.java 
> 5c75cc8cae53edfa069c85c37ebad34774682081 
>   src/main/java/org/apache/aurora/scheduler/async/Preemptor.java 
> e9f251508257cd7287ff00773e0073a3cd130df8 
>   src/main/java/org/apache/aurora/scheduler/base/JobKeys.java 
> a76c3fac71b35115064fba6644cff0066fd9e630 
>   src/main/java/org/apache/aurora/scheduler/base/Query.java 
> eded7a59eb394748b93d7fbc085a1bdf64b043cc 
>   
> src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
>  865742171c11fbe5cf1469a69dd7258ec1be28c2 
>   src/main/java/org/apache/aurora/scheduler/http/Utilization.java 
> a0cb7bf56aeb7edd92b25d8d69a739d87452777a 
>   src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java 
> 8c20ab6f2bebf1d1c0f91fed3f1e48361cdf45d6 
>   src/main/java/org/apache/aurora/scheduler/storage/mem/MemTaskStore.java 
> 37176237fac336413267f3c8bb4e1b9a6255150c 
>   
> src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
>  5dcae4a6132026504cf02093ad4c68ab521e4ab8 
>   src/main/python/apache/aurora/client/api/instance_watcher.py 
> b390aa8993205f1f6938f8c295e3c16a0bf4df6d 
>   src/main/python/apache/aurora/client/api/sla.py 
> b9b64680b15f5395ed6aca681b9b1c30ffe2822c 
>   src/main/python/apache/aurora/client/cli/task.py 
> c41484bdc27266443bc4e139e1ebb362a59be0f9 
>   src/main/python/apache/aurora/client/commands/admin.py 
> deee0250f3ba9837feeb92acc654f5b3b68b4e0f 
>   src/main/python/apache/aurora/client/commands/core.py 
> 58f419e674f1a9a0ae9da6faa2e39c8167bab597 
>   src/main/python/apache/aurora/client/commands/ssh.py 
> d2b8bf675556b924d3d63b545d036dc48a081486 
>   src/main/python/apache/aurora/config/thrift.py 
> 288fb40f65629c8fd4eb7d92c8bf02369237de3b 
>   src/main/python/apache/aurora/executor/aurora_executor.py 
> 2c6423d096656f426a4385f4edef6875ebad7049 
>   src/main/python/apache/aurora/executor/common/announcer.py 
> 74b2114d1ede7a4a4a68b78144f9b08bc994bd87 
>   src/main/python/apache/aurora/executor/thermos_task_runner.py 
> 7b346e253677ee9b42c57782f7f67ff63b6a0083 
>   src/main/resources/scheduler/assets/js/controllers.js 
> 7e9037ee921b009dc2b7c5adcf057bedebb01632 
>   src/main/resources/scheduler/assets/js/services.js 
> b744f375411e09b7f776e4a05ee5961227143439 
>   src/main/thrift/org/apache/aurora/gen/api.thrift 
> 8794731f4b3f1033588bdfa33c292e4796319a2a 
>   src/test/java/org/apache/aurora/scheduler/MesosTaskFactoryImplTest.java 
> e96974764844b5d1a3a05f6996075fccee209594 
>   src/test/java/org/apache/aurora/scheduler/TaskVarsTest.java 
> 371ae87f5954fa5f092db1f6d21e2291d7576173 
>   src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
> 606c4434b7158220ccf1403b6deac939021fee31 
>   src/test/java/org/apache/aurora/scheduler/async/GcExecutorLauncherTest.java 
> f2d153f446247032ad9d8d173fb70870dbfdcca1 
>   src/test/java/org/apache/aurora/scheduler/async/PreemptorImplTest.java 
> 8ee84cda8670d117e2efa2d1a114da6f0d8315d6 
>   
> src/test/java/org/apache/aurora/scheduler/async/RescheduleCalculatorImplTest.java
>  131bd826dfe47f40f3c27f29c095ed42953e316c 
>   src/test/java/org/apache/aurora/scheduler/async/TaskGroupsTest.java 
> efdde15939b2a851e38be53cceab395cc2cd82a1 
>   src/test/java/org/apache/aurora/scheduler/async/TaskHistoryPrunerTest.java 
> 53d2c6bb78ad08a84639c1ecd48ba64d1

Re: Review Request 26394: Deprecating Identity struct (renaming fields).

2014-10-09 Thread Bill Farner


> On Oct. 9, 2014, 12:05 a.m., Bill Farner wrote:
> > Thanks again for leading this - i'm very happy to see momentum on removing 
> > `Identity`.
> > 
> > Stepping back - i wonder if we should re-evaluate the way we do field 
> > deprecations now that we've established a protocol with JIRA and releases.  
> > This might mean we don't need to do the `DEPRECATED` mangling.  What do you 
> > think?
> 
> Maxim Khutornenko wrote:
> I still think visual code reminder is quite beneficial in avoiding the 
> use of deprecated fields. The between-release time is just too long to 
> remember what needs to be avoided when coding far away from thrift schema. 
> Besides, it helps validate the concept and will facilitate later removal.

Right, they're basically our poor-man's deprecation warnings.  However, this 
field and file (api.thrift) makes up part of our API, and any consumers of the 
thrift definition, or `/apibeta` will be broken by it.  I don't think we can 
get around a formal deprecation process for field renames.


- Bill


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26394/#review55907
---


On Oct. 8, 2014, 11:39 p.m., Maxim Khutornenko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26394/
> ---
> 
> (Updated Oct. 8, 2014, 11:39 p.m.)
> 
> 
> Review request for Aurora, David McLaughlin, Mark Chu-Carroll, and Bill 
> Farner.
> 
> 
> Bugs: AURORA-84
> https://issues.apache.org/jira/browse/AURORA-84
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Part 1 of Identity struct deprecation: renaming fields.
> 
> 
> Diffs
> -
> 
>   examples/jobs/hello_world.aurora fc7877c3a60e56e301d9ee1fabd73446afca7236 
>   src/main/java/org/apache/aurora/scheduler/TaskIdGenerator.java 
> 5c75cc8cae53edfa069c85c37ebad34774682081 
>   src/main/java/org/apache/aurora/scheduler/async/Preemptor.java 
> e9f251508257cd7287ff00773e0073a3cd130df8 
>   src/main/java/org/apache/aurora/scheduler/base/JobKeys.java 
> a76c3fac71b35115064fba6644cff0066fd9e630 
>   src/main/java/org/apache/aurora/scheduler/base/Query.java 
> eded7a59eb394748b93d7fbc085a1bdf64b043cc 
>   
> src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
>  865742171c11fbe5cf1469a69dd7258ec1be28c2 
>   src/main/java/org/apache/aurora/scheduler/http/Utilization.java 
> a0cb7bf56aeb7edd92b25d8d69a739d87452777a 
>   src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java 
> 8c20ab6f2bebf1d1c0f91fed3f1e48361cdf45d6 
>   src/main/java/org/apache/aurora/scheduler/storage/mem/MemTaskStore.java 
> 37176237fac336413267f3c8bb4e1b9a6255150c 
>   
> src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
>  5dcae4a6132026504cf02093ad4c68ab521e4ab8 
>   src/main/python/apache/aurora/client/api/instance_watcher.py 
> b390aa8993205f1f6938f8c295e3c16a0bf4df6d 
>   src/main/python/apache/aurora/client/api/sla.py 
> b9b64680b15f5395ed6aca681b9b1c30ffe2822c 
>   src/main/python/apache/aurora/client/cli/task.py 
> c41484bdc27266443bc4e139e1ebb362a59be0f9 
>   src/main/python/apache/aurora/client/commands/admin.py 
> deee0250f3ba9837feeb92acc654f5b3b68b4e0f 
>   src/main/python/apache/aurora/client/commands/core.py 
> 58f419e674f1a9a0ae9da6faa2e39c8167bab597 
>   src/main/python/apache/aurora/client/commands/ssh.py 
> d2b8bf675556b924d3d63b545d036dc48a081486 
>   src/main/python/apache/aurora/config/thrift.py 
> 288fb40f65629c8fd4eb7d92c8bf02369237de3b 
>   src/main/python/apache/aurora/executor/aurora_executor.py 
> 2c6423d096656f426a4385f4edef6875ebad7049 
>   src/main/python/apache/aurora/executor/common/announcer.py 
> 74b2114d1ede7a4a4a68b78144f9b08bc994bd87 
>   src/main/python/apache/aurora/executor/thermos_task_runner.py 
> 7b346e253677ee9b42c57782f7f67ff63b6a0083 
>   src/main/resources/scheduler/assets/js/controllers.js 
> 7e9037ee921b009dc2b7c5adcf057bedebb01632 
>   src/main/resources/scheduler/assets/js/services.js 
> b744f375411e09b7f776e4a05ee5961227143439 
>   src/main/thrift/org/apache/aurora/gen/api.thrift 
> 8794731f4b3f1033588bdfa33c292e4796319a2a 
>   src/test/java/org/apache/aurora/scheduler/MesosTaskFactoryImplTest.java 
> e96974764844b5d1a3a05f6996075fccee209594 
>   src/test/java/org/apache/aurora/scheduler/TaskVarsTest.java 
> 371ae87f5954fa5f092db1f6d21e2291d7576173 
>   src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
> 606c4434b7158220ccf1403b6deac939021fee31 
>   src/test/java/org/apache/aurora/scheduler/async/GcExecutorLauncherTest.java 
> f2d153f446247032ad9d8d173fb70870dbfdcca1 
>   src/test/java/org/apache/aurora/scheduler/async/PreemptorImplTest.java 
> 8ee84cda8670d117e2efa2d1a114da6f0d8315d6 
>   
> src/test/java/org/apach