Return-Path: X-Original-To: apmail-aurora-reviews-archive@minotaur.apache.org Delivered-To: apmail-aurora-reviews-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 2D6D718805 for ; Thu, 23 Jul 2015 05:59:08 +0000 (UTC) Received: (qmail 78516 invoked by uid 500); 23 Jul 2015 05:58:55 -0000 Delivered-To: apmail-aurora-reviews-archive@aurora.apache.org Received: (qmail 78465 invoked by uid 500); 23 Jul 2015 05:58:55 -0000 Mailing-List: contact reviews-help@aurora.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@aurora.apache.org Delivered-To: mailing list reviews@aurora.apache.org Received: (qmail 78436 invoked by uid 99); 23 Jul 2015 05:58:55 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 23 Jul 2015 05:58:55 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id DB54CBED0C; Thu, 23 Jul 2015 05:58:52 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============4155446764284438961==" MIME-Version: 1.0 Subject: Re: Review Request 36711: Remove all public static com.google.common.base.Functions From: "Bill Farner" To: "Bill Farner" , "Zameer Manji" Cc: "Kevin Sweeney" , "Aurora" Date: Thu, 23 Jul 2015 05:58:52 -0000 Message-ID: <20150723055852.10542.13307@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Bill Farner" X-ReviewGroup: Aurora X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/36711/ X-Sender: "Bill Farner" References: <20150722231813.10542.28011@reviews.apache.org> In-Reply-To: <20150722231813.10542.28011@reviews.apache.org> Reply-To: "Bill Farner" X-ReviewRequest-Repository: aurora --===============4155446764284438961== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36711/#review92727 ----------------------------------------------------------- Awaiting a happy review bot before i step in. - Bill Farner On July 22, 2015, 11:18 p.m., Kevin Sweeney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36711/ > ----------------------------------------------------------- > > (Updated July 22, 2015, 11:18 p.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 > 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/async/RescheduleCalculator.java 6a0c0a9b933546c1581d3e29ba53eb86c58fbc39 > src/main/java/org/apache/aurora/scheduler/async/TaskGroups.java 1580404a36bbab6b612b8188c4a649d4a77118e0 > src/main/java/org/apache/aurora/scheduler/async/TaskHistoryPruner.java 7b6c0639fba6a322529f17c9a69c3b28ea7d2432 > src/main/java/org/apache/aurora/scheduler/async/TaskScheduler.java a500e55cb52fe89917d8499a42e2a2eb5032c109 > src/main/java/org/apache/aurora/scheduler/async/preemptor/PendingTaskProcessor.java 1f1eb4c1f2f2b995d616b70ba9e3d149eea413d0 > src/main/java/org/apache/aurora/scheduler/base/JobKeys.java a5ffa5e95b301e536a84acf02817ea0c080559d0 > 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 c0d165ad34e46653dad95918e0058ebd3f2ee57f > src/main/java/org/apache/aurora/scheduler/metadata/NearestFit.java 2b194e5a761cb193a2b72419db1a1120fb52c9fe > src/main/java/org/apache/aurora/scheduler/quota/QuotaManager.java 61b8e57b0ee62a706433de9376e81b21af7e9e08 > 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 ff33d0ba01794b3308be4a8baa04b4f8dbb82d3c > 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 d0156d330deb5d862fa54513fdbb6561d1c91548 > 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 22786dedd0db014f518eb5701a985559d189adad > 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/async/TaskHistoryPrunerTest.java 6eaf3ce765c8e50b6724e40848ceb9105e1ab529 > src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerImplTest.java 45adb2efc8807e22b923cb5606d797758f063297 > src/test/java/org/apache/aurora/scheduler/stats/ResourceCounterTest.java ed834bce2f1502605a8ef83d2812eea184bcc0a9 > src/test/java/org/apache/aurora/scheduler/storage/AbstractTaskStoreTest.java 775bb71f94582020643c12fd19a15d0d6732c699 > src/test/java/org/apache/aurora/scheduler/storage/mem/StorageTransactionTest.java 4ae194c4f86b609729cfdfdaeda369733efae0b2 > src/test/java/org/apache/aurora/scheduler/thrift/ReadOnlySchedulerImplTest.java ddabb4c04ef329d372504f2fbef597be004e62f1 > src/test/java/org/apache/aurora/scheduler/updater/JobUpdaterIT.java 8b99e0fac429bcbb16bd769890dcae902e3f022e > > Diff: https://reviews.apache.org/r/36711/diff/ > > > Testing > ------- > > ./gradlew -Pq build > > > Thanks, > > Kevin Sweeney > > --===============4155446764284438961==--