----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48559/ -----------------------------------------------------------
(Updated June 17, 2016, 12:46 p.m.) Review request for Aurora and Maxim Khutornenko. Changes ------- - Moved ```TaskTestUtil``` back to ```org.apache.aurora.scheduler.base``` package - Moved back fill related logic from ```TierManagerImpl``` to ```ThriftBackfill``` - Other suggested refactorings Note: This change set still does not include client changes. Repository: aurora Description ------- AURORA-1710 Make 'tier' required and remove support for 'production' flag in Job configuration - Backfill portion Diffs (updated) ----- src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java 65f5edce74077f52e98f110fcd17b2f12d673f81 src/jmh/java/org/apache/aurora/benchmark/StatusUpdateBenchmark.java 6fd9ee21cf8e0e42e73a68cdf2d231d581278aae src/main/java/org/apache/aurora/scheduler/TierManager.java af54cab73a80a5120b1a77fd985dfbaf568d786c src/main/java/org/apache/aurora/scheduler/base/TaskTestUtil.java 3ea0992eb0a9930a4db9eb4b7fcab82689495c1f src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java 0e9562020c298e685e6c2efd18933818b03a5000 src/main/java/org/apache/aurora/scheduler/storage/backup/TemporaryStorage.java d08873c88f159eb65b582840b48b7ff604862c31 src/main/java/org/apache/aurora/scheduler/storage/db/migration/V006_PopulateTierField.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java c981a05e61cb053a05144c702c9ffafeb0af8260 src/main/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImpl.java 8eed1fc680b0c4fb27d8a353b7f804ae09058156 src/main/java/org/apache/aurora/scheduler/storage/log/ThriftBackfill.java 0a307fe8d8238c23a526d5c3ee500e1de0761703 src/test/java/org/apache/aurora/scheduler/TierManagerTest.java d4b71f8dbb674384ccbbd9e76f510d127e480e32 src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 8c8c793813e84bf7ef741f9b6b4ae1e759be1b82 src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java 2e322d217fc9dc75c51b57607a5547745206fb9f src/test/java/org/apache/aurora/scheduler/storage/backup/RecoveryTest.java e870087e3d47906559410ff76515457f4ff99ff5 src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java be1132b439948104458efdc82a6bbee43c20c4fd src/test/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImplIT.java e0cf602ead1530301b09eff60287b8fa48be63e8 src/test/java/org/apache/aurora/scheduler/storage/log/ThriftBackfillTest.java 0a2cd3d5b01c389f99fca169227aac35436d474b src/test/java/org/apache/aurora/scheduler/thrift/Fixtures.java 4f8158546f3eba8f79d653ad7a30f83d66cbce83 src/test/java/org/apache/aurora/scheduler/thrift/ReadOnlySchedulerImplTest.java 6f355d6e66c05651fa9b13356dd81b45bee52adc src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java ecdc62ae3b21b73b6a6af80bb9855867a7e965e0 src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java a54d169caebfc211035386f64169ecd983e378d9 Diff: https://reviews.apache.org/r/48559/diff/ Testing ------- Manual under Vagrant: - Deployed old scheduler (with tier backfill support), created a job without tier, upgraded scheduler, noticed that tier has been backfilled - Tried the above scenario with both -use_beta_db_task_store=true and -use_beta_db_task_store=false configuration flags - Verified that if tier is already set it would not be altered - Verified that it works both when production = 'true' and production = 'false' End to End: ``` ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh ... *** OK (All tests passed) *** mesos-master start/running, process 26886 + RETCODE=0 + restore_netrc + mv /home/vagrant/.netrc.bak /home/vagrant/.netrc + true Connection to 127.0.0.1 closed. real 17m53.514s user 0m1.443s sys 0m0.624s ``` Thanks, Mehrdad Nurolahzade