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

Ship it!


Ship It!

- Bill Farner


On July 23, 2015, 7:07 a.m., Kevin Sweeney wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36711/
> -----------------------------------------------------------
> 
> (Updated July 23, 2015, 7:07 a.m.)
> 
> 
> Review request for Aurora, Bill Farner and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Remove all public static com.google.common.base.Functions
> 
> This change removes all public static Functions and either replaces them with 
> static methods or inlines them (if they could have been replaced by method 
> references). The benefit of this change is that these library functions are 
> now usable as `com.google.common.base.Function`s (in the case of guava 
> methods), `java.util.function.Function`s (in the case of the Java 8 standard 
> library), and directly (by callers that were previously invoking them via 
> `.apply`).
> 
> This change was almost entirely IDE-driven, with the exception of the direct 
> calls (IntelliJ inlines `Tasks.ASSIGNED_TO_ID.apply(assignedTask)` as 
> `((Function<IAssignedTask, String) IAssignedTask::getId).apply(assignedTask)` 
> rather than `assignedTask.getId()`).
> 
> Since overloads work now and we generally don't repeat the type in the method 
> name, some names were shortened (for example `Tasks.SCHEDULED_TO_INSTANCE_ID` 
> was renamed to `Tasks#getInstanceId(ScheduledTask)`.
> 
> I chose to only include `public static` Functions in this review to avoid a 
> diff size explosion.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/Protobufs.java 
> 7f1789531bc7fc83dba6e19b39e74f3333a7232a 
>   src/main/java/org/apache/aurora/scheduler/base/JobKeys.java 
> 38d2863632e47e0d08c60c59a5d20222b5ebe66a 
>   src/main/java/org/apache/aurora/scheduler/base/TaskGroupKey.java 
> 47e4d48ab4c82b2770a6817666dbd24c3e387b48 
>   src/main/java/org/apache/aurora/scheduler/base/Tasks.java 
> 52d37e255edbabc9874591b7a4cb3e096ac51ecc 
>   src/main/java/org/apache/aurora/scheduler/filter/AttributeAggregate.java 
> ae7458c7ea1bb624d0d10539e2ef9f34a77411de 
>   src/main/java/org/apache/aurora/scheduler/http/Maintenance.java 
> 303f05cbec7a7134b39ad1d15af3ee5e7a793e0c 
>   src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java 
> c91c0ea3933e725e3aee5faeb3309f0deba238e2 
>   src/main/java/org/apache/aurora/scheduler/metadata/NearestFit.java 
> 2b194e5a761cb193a2b72419db1a1120fb52c9fe 
>   
> src/main/java/org/apache/aurora/scheduler/preemptor/PendingTaskProcessor.java 
> f1b075a464368adb73be18f19feb938287921017 
>   src/main/java/org/apache/aurora/scheduler/pruning/TaskHistoryPruner.java 
> e6e76cd1007f06bfa089b7171ef326fe52e492c9 
>   src/main/java/org/apache/aurora/scheduler/quota/QuotaManager.java 
> 61b8e57b0ee62a706433de9376e81b21af7e9e08 
>   
> src/main/java/org/apache/aurora/scheduler/scheduling/RescheduleCalculator.java
>  bfc23cd88a2a4ce560568ff28d96d8b78eeb6c9e 
>   src/main/java/org/apache/aurora/scheduler/scheduling/TaskGroups.java 
> 3f262bf99b4723cba0df05675db5e57976dc7a0f 
>   src/main/java/org/apache/aurora/scheduler/scheduling/TaskScheduler.java 
> 04e506368e4b6360b35ab8843b318891806b1853 
>   src/main/java/org/apache/aurora/scheduler/sla/MetricCalculator.java 
> 7dca57490fc521fd7497b991aa8778f68b92b03b 
>   src/main/java/org/apache/aurora/scheduler/sla/SlaAlgorithm.java 
> 42967dff996710d3ec1172f481e1f1fb99a20b03 
>   src/main/java/org/apache/aurora/scheduler/sla/SlaGroup.java 
> a1d6c9d6857d2bacfcc7e4719ef2b42b66a9b470 
>   src/main/java/org/apache/aurora/scheduler/state/MaintenanceController.java 
> 441f828da2466288136124f2184aae0d5507848b 
>   src/main/java/org/apache/aurora/scheduler/state/StateManagerImpl.java 
> 08844d047e021631e8aad359bcb1a6f698425b90 
>   src/main/java/org/apache/aurora/scheduler/state/TaskStateMachine.java 
> 4f453e43e21f170293449853f28be49ba0fcb5f8 
>   src/main/java/org/apache/aurora/scheduler/stats/ResourceCounter.java 
> ca0c0b56a6cc8f7df782763cdc6ff77447cdd09a 
>   src/main/java/org/apache/aurora/scheduler/storage/AttributeStore.java 
> 36bdc819a57cc59c6b8eff7d9d0261408aa1fc8a 
>   
> src/main/java/org/apache/aurora/scheduler/storage/backup/TemporaryStorage.java
>  fc7c79265aff3520cb875c25cf177ffc2f8e0ec9 
>   
> src/main/java/org/apache/aurora/scheduler/storage/log/WriteAheadStorage.java 
> 0b5530478b6b43b87dbf095a3b4c53dc33eaddcf 
>   src/main/java/org/apache/aurora/scheduler/storage/mem/MemTaskStore.java 
> 42c11192649c1f068efa8e118818d4b27273d809 
>   src/main/java/org/apache/aurora/scheduler/thrift/ReadOnlySchedulerImpl.java 
> 92c970c34ca9dc4f052760e5a3d3770a089d9a67 
>   
> src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
>  062cce06564c1c0cccee8a772ace8230dcff1fbe 
>   src/main/java/org/apache/aurora/scheduler/updater/InstanceUpdater.java 
> 722dba4f3dc98efeed7128bfa1f2ae02f4670972 
>   src/main/java/org/apache/aurora/scheduler/updater/JobDiff.java 
> ab540a59b2297cec76190f4df10cc816d9e1ea97 
>   
> src/main/java/org/apache/aurora/scheduler/updater/JobUpdateEventSubscriber.java
>  134cd5429512338d0a464b1f3ffa557ca99d5713 
>   
> src/test/java/org/apache/aurora/scheduler/pruning/TaskHistoryPrunerTest.java 
> 461c4d0cc26d70f0a03130581da666bf2c561bb0 
>   
> src/test/java/org/apache/aurora/scheduler/scheduling/TaskSchedulerImplTest.java
>  b964f812d20a9d9473f719c34545ff7e58a65559 
>   src/test/java/org/apache/aurora/scheduler/stats/ResourceCounterTest.java 
> a9c6d1db1d153657e7f139d81aceb560108d3de1 
>   
> src/test/java/org/apache/aurora/scheduler/storage/AbstractTaskStoreTest.java 
> 55ebcb89e9201f23430dcfae3af006ec1a7732bf 
>   
> src/test/java/org/apache/aurora/scheduler/storage/mem/StorageTransactionTest.java
>  791ec1446c2b4429b0b953e8639319622dd02c51 
>   
> src/test/java/org/apache/aurora/scheduler/thrift/ReadOnlySchedulerImplTest.java
>  ddabb4c04ef329d372504f2fbef597be004e62f1 
>   src/test/java/org/apache/aurora/scheduler/updater/JobUpdaterIT.java 
> 0d0ee9499476d84e5e6f937f8ce7320185b02a7a 
> 
> Diff: https://reviews.apache.org/r/36711/diff/
> 
> 
> Testing
> -------
> 
> ./gradlew -Pq build
> 
> 
> Thanks,
> 
> Kevin Sweeney
> 
>

Reply via email to