----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56265/#review164267 -----------------------------------------------------------
Ship it! Should we chase this patch with a source-wide refactoring from `slave` to `agent`? - Mehrdad Nurolahzade On Feb. 3, 2017, 11:38 a.m., Zameer Manji wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56265/ > ----------------------------------------------------------- > > (Updated Feb. 3, 2017, 11:38 a.m.) > > > Review request for Aurora, Mehrdad Nurolahzade and Stephan Erb. > > > Bugs: AURORA-1886 > https://issues.apache.org/jira/browse/AURORA-1886 > > > Repository: aurora > > > Description > ------- > > This is the first step in moving Aurora to the V1 API from Mesos. This patch > moves most of the code to v1 Protobufs. This means all peices of code that do > not interact with Mesos now handle only v1 Protobufs. > > Classes that interact with Mesos directly are: > > * `org.apache.aurora.scheduler.mesos.SchedulerDriverService` > * `org.apache.aurora.scheduler.mesos.MesosSchedulerImpl` > * `org.apache.aurora.scheduler.mesos.DriverFactoryImpl` > > These classes handle unversioned Protobufs and use the `ProtosConversion` > class > to convert them to v1 Protobufs that can be safely passed to the rest of the > code. > > > Diffs > ----- > > src/jmh/java/org/apache/aurora/benchmark/Offers.java > 144f47ea07222d8a972f311b5eaf407fcc502a21 > src/jmh/java/org/apache/aurora/benchmark/StatusUpdateBenchmark.java > a649239c80e031c6d4e3d0770a5d4728f897a94b > src/jmh/java/org/apache/aurora/benchmark/fakes/FakeDriver.java > 9674c76120cf7b748ac4c8ace1da84483442996e > src/jmh/java/org/apache/aurora/benchmark/fakes/FakeOfferManager.java > fbd24ea4a58e28c14a343170de137c0e0ae437a2 > src/main/java/org/apache/aurora/scheduler/HostOffer.java > ad30bf978ae5aa278fa9b5e01294c43892b08762 > src/main/java/org/apache/aurora/scheduler/SchedulerModule.java > 2ec3967ddb1d470cf681de062a6400f647978185 > src/main/java/org/apache/aurora/scheduler/TaskStatusHandler.java > 3e132ee8595b9b771adcc9580af213c1a4439e69 > src/main/java/org/apache/aurora/scheduler/TaskStatusHandlerImpl.java > 6afafe875fc31982bc1afcebe40e4953172a7984 > src/main/java/org/apache/aurora/scheduler/base/Conversions.java > d08b6cf1c7f9e49fefd3b560aeba9bb4a53a20fc > src/main/java/org/apache/aurora/scheduler/base/Numbers.java > 1b278e29555cf4dbaba63b7b87e8b21ae78f9786 > src/main/java/org/apache/aurora/scheduler/base/TaskTestUtil.java > aba73005019c13ac943be9c53ac58c8ce5bfba94 > > src/main/java/org/apache/aurora/scheduler/configuration/executor/ExecutorConfig.java > 32bafb2b3ef4ea64fa0a5e3ffc43b47361fd3358 > > src/main/java/org/apache/aurora/scheduler/configuration/executor/ExecutorModule.java > 0d6a8c9acfbf4a0838fa580da09ce4c50a1b0761 > > src/main/java/org/apache/aurora/scheduler/configuration/executor/ExecutorSettingsLoader.java > 18ff2e2baebe1784bc31bc7f3a685282a2bed915 > > src/main/java/org/apache/aurora/scheduler/configuration/executor/Executors.java > 6ac2d006c45393592e3e8b4ed94f99500f6518b2 > src/main/java/org/apache/aurora/scheduler/events/PubsubEvent.java > 70b5470b9dad1af838b5222cae5ac86487e2f2e4 > > src/main/java/org/apache/aurora/scheduler/mesos/CommandLineDriverSettingsModule.java > 36c69be9ccf152c5d74466a08ddca8029b4a13c9 > src/main/java/org/apache/aurora/scheduler/mesos/Driver.java > bb208ea7f77402a07aef162930288219595054a2 > src/main/java/org/apache/aurora/scheduler/mesos/DriverFactory.java > 92d8924aa182ae272fd394ce68dcec0bf35a118d > src/main/java/org/apache/aurora/scheduler/mesos/DriverFactoryImpl.java > a5f5e9940c467b7f8584254b4bfefa3b5d059abe > src/main/java/org/apache/aurora/scheduler/mesos/DriverSettings.java > 85d471ff88618665a377182b5b1c208278e3ccea > src/main/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImpl.java > 7b2614f3f2b552235cf93ca285bcf9c999457e46 > src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java > 3d5c3bd139206e970811aa95bd74b78987bb9cfe > src/main/java/org/apache/aurora/scheduler/mesos/ProtosConversion.java > PRE-CREATION > src/main/java/org/apache/aurora/scheduler/mesos/SchedulerDriverService.java > c89be79f5610a56477fbbd4ae8b2475a682456b6 > src/main/java/org/apache/aurora/scheduler/mesos/TaskStatusStats.java > 5573638f0a18ae6cdd84f648160d0e63c27f7510 > src/main/java/org/apache/aurora/scheduler/mesos/TestExecutorSettings.java > fe54411906c2d5546266488579e30c09af5bdc56 > src/main/java/org/apache/aurora/scheduler/offers/OfferManager.java > 2b12696d0218a48d71e89df4d3a38c6bf84c74c7 > > src/main/java/org/apache/aurora/scheduler/preemptor/PendingTaskProcessor.java > dad1cbbe6d3d4a16fda5530c56480c1b55a81324 > src/main/java/org/apache/aurora/scheduler/preemptor/Preemptor.java > 6b807e0335dfc6295b3ea96ebcdab858bb40f57d > > src/main/java/org/apache/aurora/scheduler/reconciliation/TaskReconciler.java > 003fe5389e50c97bde3bf4fe62b5a43f5caa1ebe > src/main/java/org/apache/aurora/scheduler/resources/AcceptedOffer.java > c100e80303036f6f384928c915c8bea35b4e09d1 > > src/main/java/org/apache/aurora/scheduler/resources/MesosResourceConverter.java > 31235069410170d1c9d6583a91714836a62dd407 > src/main/java/org/apache/aurora/scheduler/resources/ResourceManager.java > d751d017522c682a1373932e286ee7b4c447e2aa > src/main/java/org/apache/aurora/scheduler/resources/ResourceMapper.java > dc57d57d2b7ed885db562a94132b85771b4ae084 > src/main/java/org/apache/aurora/scheduler/resources/ResourceType.java > 178466f1e4e640929bce64e8215118f336b7d0eb > src/main/java/org/apache/aurora/scheduler/state/StateManager.java > d395104af84477b03649ddd65276fbd1dc1e9210 > src/main/java/org/apache/aurora/scheduler/state/StateManagerImpl.java > 7b70c41815792ca085436565a2a6267d04e99980 > src/main/java/org/apache/aurora/scheduler/state/TaskAssigner.java > a030cebea13d92766584643ddda33c50dec7bdbf > src/main/java/org/apache/aurora/scheduler/storage/AttributeStore.java > 981bdcf665fdfa710608c57e7b7817d286dab373 > src/test/java/org/apache/aurora/ProtobufsTest.java > 6c388854ac31ba8db809efed4277878c25b085f8 > src/test/java/org/apache/aurora/scheduler/TaskStatusHandlerImplTest.java > b28d703ab58f9c47a27ad6d1c407af62b17012d4 > src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java > 29a3b4a57925f31e59a49d4bfa630e724cadcb04 > src/test/java/org/apache/aurora/scheduler/app/local/FakeMaster.java > 965bfa7702a6d7308c767f13b2740f1ad94248a0 > src/test/java/org/apache/aurora/scheduler/app/local/LocalSchedulerMain.java > 80fd223b241b43142fb346d3a0a1314f8a04705e > > src/test/java/org/apache/aurora/scheduler/app/local/simulator/ClusterSimulatorModule.java > 8c0d179afac37165a26e4cc451acbc049a70d6d6 > > src/test/java/org/apache/aurora/scheduler/app/local/simulator/FakeSlaves.java > d8724e4aad44b609383fcbaac0e62c7ff9e0bdd1 > > src/test/java/org/apache/aurora/scheduler/app/local/simulator/events/OfferAccepted.java > 2fd9c5ee73595177972bf980ba89a98b5a8e80c7 > src/test/java/org/apache/aurora/scheduler/base/ConversionsTest.java > 9f94a98b2c4d59b8d1ee75700e29de649d59108b > > src/test/java/org/apache/aurora/scheduler/configuration/executor/ExecutorModuleTest.java > d597b13b95e25e768d7c5010e4f6888c8e2d9c5a > > src/test/java/org/apache/aurora/scheduler/configuration/executor/ExecutorSettingsLoaderTest.java > b8cfbaa3f3e5e03a7e405d5eeca12bb9b7587207 > src/test/java/org/apache/aurora/scheduler/http/OffersTest.java > add0eb8f861a36551ed87ba13a75b82c5cd5bdfa > > src/test/java/org/apache/aurora/scheduler/mesos/CommandLineDriverSettingsModuleTest.java > ffc960c2698013c93765765dc7fc4c6c624e3653 > src/test/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImplTest.java > ed48934c3954a4a092158f1f25f98c0eb7782b37 > > src/test/java/org/apache/aurora/scheduler/mesos/MesosTaskFactoryImplTest.java > 234acc13164a05b6886bcdd2bb9296397ec0b903 > src/test/java/org/apache/aurora/scheduler/mesos/ProtosConversionTest.java > PRE-CREATION > > src/test/java/org/apache/aurora/scheduler/mesos/SchedulerDriverServiceTest.java > 35620a144b5eec5ebe7633d8eb4c4e554868e5ea > src/test/java/org/apache/aurora/scheduler/mesos/TaskStatusStatsTest.java > a48c85bcedb537f8081fb00abdca6e210083e7cf > src/test/java/org/apache/aurora/scheduler/offers/OfferManagerImplTest.java > fb8bd85be93280a9b8366b1398a1fa0a5817bb55 > src/test/java/org/apache/aurora/scheduler/offers/Offers.java > c0899b069fe6f7e8160344f894ba00039e95852b > > src/test/java/org/apache/aurora/scheduler/preemptor/PendingTaskProcessorTest.java > 5c115f3f7840b93ab05ad9aeee4dff417e5c5a5e > > src/test/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilterTest.java > 64da234aa6b0569ded365badd06bd0b780351c4d > src/test/java/org/apache/aurora/scheduler/preemptor/PreemptorImplTest.java > 3b932c9858cd6222cb61f63800566226bf1acf4d > > src/test/java/org/apache/aurora/scheduler/reconciliation/TaskReconcilerTest.java > cc6f44df1afc0c8ebb1b74e13a930ce012503f09 > src/test/java/org/apache/aurora/scheduler/resources/AcceptedOfferTest.java > 2777d7276d55cc0fe75e7470d0dc8182796d67f4 > > src/test/java/org/apache/aurora/scheduler/resources/MesosResourceConverterTest.java > 2f67ef4f40ebbb9910c806fa8d48933b8d6ed272 > src/test/java/org/apache/aurora/scheduler/resources/PortMapperTest.java > 244c9de966311789e97a5fb1a5ab6625c232c7ac > > src/test/java/org/apache/aurora/scheduler/resources/ResourceManagerTest.java > 133cd23427485ea329ef10489e5d4a0029cb54cd > src/test/java/org/apache/aurora/scheduler/resources/ResourceTestUtil.java > 7dc8c179736fab93ca03e174b16e104ba0118127 > src/test/java/org/apache/aurora/scheduler/state/StateManagerImplTest.java > fbcc4003f7647580b859f4dc08a16c3e471a8f18 > src/test/java/org/apache/aurora/scheduler/state/TaskAssignerImplTest.java > f8bb25b612ae42748205901dd4b8cfcdedf5b136 > src/test/java/org/apache/aurora/scheduler/stats/AsyncStatsModuleTest.java > 75bac6f238a619c41903007babd4a6e20aaeaa1e > src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java > 86861e1bd4ca5b920808abf419bc13f83961b836 > > Diff: https://reviews.apache.org/r/56265/diff/ > > > Testing > ------- > > > Thanks, > > Zameer Manji > >