From reviews-return-20520-archive-asf-public=cust-asf.ponee.io@aurora.apache.org Sat Jan 20 04:16:11 2018 Return-Path: X-Original-To: archive-asf-public@eu.ponee.io Delivered-To: archive-asf-public@eu.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by mx-eu-01.ponee.io (Postfix) with ESMTP id B289D180607 for ; Sat, 20 Jan 2018 04:16:11 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id 9D9F7160C49; Sat, 20 Jan 2018 03:16:11 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id BD935160C36 for ; Sat, 20 Jan 2018 04:16:10 +0100 (CET) Received: (qmail 42479 invoked by uid 500); 20 Jan 2018 03:16:10 -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 42468 invoked by uid 99); 20 Jan 2018 03:16:09 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd2-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 20 Jan 2018 03:16:09 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd2-us-west.apache.org (ASF Mail Server at spamd2-us-west.apache.org) with ESMTP id 24D161A0664; Sat, 20 Jan 2018 03:16:09 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd2-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 2.541 X-Spam-Level: ** X-Spam-Status: No, score=2.541 tagged_above=-999 required=6.31 tests=[HTML_MESSAGE=2, KAM_LAZY_DOMAIN_SECURITY=1, KAM_LOTSOFHASH=0.25, RCVD_IN_DNSWL_LOW=-0.7, T_RP_MATCHES_RCVD=-0.01, URIBL_BLOCKED=0.001] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd2-us-west.apache.org [10.40.0.9]) (amavisd-new, port 10024) with ESMTP id 7jTjqvUyQVbT; Sat, 20 Jan 2018 03:16:06 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id E08075F23C; Sat, 20 Jan 2018 03:16:05 +0000 (UTC) Received: from reviews.apache.org (unknown [10.41.0.12]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id 09BDCE0E34; Sat, 20 Jan 2018 03:16:05 +0000 (UTC) Received: from reviews-vm2.apache.org (localhost [IPv6:::1]) by reviews.apache.org (ASF Mail Server at reviews-vm2.apache.org) with ESMTP id B7DA8C40A0D; Sat, 20 Jan 2018 03:16:02 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============3716683568884783217==" MIME-Version: 1.0 Subject: Re: Review Request 65233: Allow for injection of custom OfferSets, removed OfferOrder and OfferSelector From: Aurora ReviewBot To: Bill Farner , David McLaughlin , Stephan Erb Cc: Aurora ReviewBot , Aurora , Jordan Ly Date: Sat, 20 Jan 2018 03:16:01 -0000 Message-ID: <20180120031601.64727.51147@reviews-vm2.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Aurora ReviewBot X-ReviewGroup: Aurora X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/65233/ X-Sender: Aurora ReviewBot X-ReviewBoard-ShipIt: 1 References: <20180120013105.64895.24459@reviews-vm2.apache.org> In-Reply-To: <20180120013105.64895.24459@reviews-vm2.apache.org> X-ReviewBoard-Diff-For: src/test/java/org/apache/aurora/scheduler/scheduling/FirstFitOfferSelectorTest.java X-ReviewBoard-Diff-For: src/main/java/org/apache/aurora/scheduler/scheduling/OfferSelector.java X-ReviewBoard-Diff-For: src/main/java/org/apache/aurora/scheduler/offers/OfferSet.java X-ReviewBoard-Diff-For: src/main/java/org/apache/aurora/scheduler/scheduling/FirstFitOfferSelector.java X-ReviewBoard-Diff-For: src/main/java/org/apache/aurora/scheduler/scheduling/FirstFitOfferSelectorModule.java X-ReviewBoard-Diff-For: src/main/java/org/apache/aurora/scheduler/offers/OfferSetImpl.java Reply-To: Aurora ReviewBot X-ReviewRequest-Repository: aurora --===============3716683568884783217== 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/65233/#review195870 ----------------------------------------------------------- Ship it! Master (c4c55ff) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Jan. 19, 2018, 5:31 p.m., Jordan Ly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65233/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2018, 5:31 p.m.) > > > Review request for Aurora, David McLaughlin, Stephan Erb, and Bill Farner. > > > Repository: aurora > > > Description > ------- > > The goal of this patch is to provide a more reasonable abstraction for custom scheduling. > > Currently, we have `OfferSelector`, `OfferOrder`, and recently I proposed `FilterableOfferCollection` (https://reviews.apache.org/r/65225/). These were all created in order to provide more customization within the scheduling loop. However, they suffer from being leaky and too disparate. This patch hopes to combine all of those components into a single `OfferSet` which can be injected and used by HostOffers. This interface allows for custom scheduling logic to be co-located with custom data structures for `offers` as opposed to being passed around as different components. > > The following options will be removed from the last 0.19 to now: > ``` > -offer_order_modules > ``` > > > Diffs > ----- > > RELEASE-NOTES.md 3053e543793669601ec9132adae376bcdce64c32 > docs/reference/scheduler-configuration.md f697b6f1f5332ac9a9feade48317ecf5818e0c92 > src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java f44cb61f5d1805aeb0f98135453dbc82f3bc9ad4 > src/main/java/org/apache/aurora/scheduler/config/CliOptions.java e4e535824adb86c98c6173a20f97a9a90b08483a > src/main/java/org/apache/aurora/scheduler/offers/HostOffers.java 2ea7a01085b87c8ed6765537a8005e2349784ab0 > src/main/java/org/apache/aurora/scheduler/offers/OfferManager.java 8f9e33d81be9087821784a8a08079a1736d9cb63 > src/main/java/org/apache/aurora/scheduler/offers/OfferManagerModule.java de16c14c887d4225e0629b1580eb5e740798f1f7 > src/main/java/org/apache/aurora/scheduler/offers/OfferOrderBuilder.java 1260ef19506acb8e8a937d4fd7b7152361bd3c40 > src/main/java/org/apache/aurora/scheduler/offers/OfferSet.java PRE-CREATION > src/main/java/org/apache/aurora/scheduler/offers/OfferSetImpl.java PRE-CREATION > src/main/java/org/apache/aurora/scheduler/offers/OfferSettings.java 1e36b2c3094e99f05aa4a4f098a48df2293b4320 > src/main/java/org/apache/aurora/scheduler/scheduling/FirstFitOfferSelector.java 5f4f452cb133e2be73f881700a54ba4c8e60a35a > src/main/java/org/apache/aurora/scheduler/scheduling/FirstFitOfferSelectorModule.java 4d36487f430483444314a8cefd133e7c766075de > src/main/java/org/apache/aurora/scheduler/scheduling/OfferSelector.java 6f5d55c5a586ac923b5f375d1935ec2049673ec0 > src/main/java/org/apache/aurora/scheduler/scheduling/TaskAssignerImpl.java ec416cc12b2fb4a11a273c77dbe72b79f96fa7e1 > src/main/java/org/apache/aurora/scheduler/scheduling/TaskAssignerImplModule.java d101a497896ba3b8ca7984a63f861494cbddbb35 > src/test/java/org/apache/aurora/scheduler/config/CommandLineTest.java c84fad3b615c49ca7527ae3f7fa116aa8f90ff0c > src/test/java/org/apache/aurora/scheduler/offers/OfferManagerImplTest.java 28224a5c587b235ccc86a286e796eeca66929232 > src/test/java/org/apache/aurora/scheduler/scheduling/FirstFitOfferSelectorTest.java e6b2b7465fb4fd432cda50d1e714dfc587764689 > src/test/java/org/apache/aurora/scheduler/scheduling/TaskAssignerImplTest.java f84941edd96d974a6c2d7e087324bc8fbbac3b0a > > > Diff: https://reviews.apache.org/r/65233/diff/6/ > > > Testing > ------- > > `./gradlew test` > `./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh` > > Testing injection of a custom `OfferSet` onto a test cluster. > > > Thanks, > > Jordan Ly > > --===============3716683568884783217==--