aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David McLaughlin <da...@dmclaughlin.com>
Subject Re: Review Request 59039: Add the ability to customize scheduling logic.
Date Tue, 09 May 2017 21:42:58 GMT

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

(Updated May 9, 2017, 9:42 p.m.)


Review request for Aurora, Santhosh Kumar Shanmugham, Stephan Erb, and Zameer Manji.


Changes
-------

Added code sample of how this can be used.


Bugs: AURORA-1920
    https://issues.apache.org/jira/browse/AURORA-1920


Repository: aurora


Description
-------

Add the ability to replace the first-fit scheduling algorithm and associated first-fit preemption
logic.

See design/proposal document here: https://docs.google.com/document/d/1fVHLt9AF-YbOCVCDMQmi5DATVusn-tqY8DldKbjVEm0/edit?usp=sharing


Diffs
-----

  src/main/java/org/apache/aurora/scheduler/preemptor/PendingTaskProcessorModule.java PRE-CREATION

  src/main/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilterModule.java PRE-CREATION

  src/main/java/org/apache/aurora/scheduler/preemptor/PreemptorModule.java 66d20290d381c041ac2f28903d6ba1468b352172

  src/main/java/org/apache/aurora/scheduler/state/FirstFitTaskAssignerModule.java PRE-CREATION

  src/main/java/org/apache/aurora/scheduler/state/StateModule.java 0186484208965c5b8942ec76eb078b30190c7d57

  src/test/java/org/apache/aurora/scheduler/preemptor/PreemptorModuleTest.java da064e3589951d4f4cbff96d1ed89d9d7e4d3882



Diff: https://reviews.apache.org/r/59039/diff/1/


Testing (updated)
-------

./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh

I've used this in one of our large-scale test clusters to plug in a custom task assigner.
We were able to remove our modifications to the OSS code-base and use this mechanism to do
what we need. Here is a rough gist of what it looks like:
https://gist.github.com/DavidMcLaughlin/5d5472cde15343aef8705478f644ddb9


Thanks,

David McLaughlin


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message