aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <>
Subject Re: Review Request 42565: Remove support for adding guice modules via command line arguments.
Date Wed, 20 Jan 2016 20:21:07 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Jan. 20, 2016, 12:21 p.m.)

Review request for Aurora, John Sirois and Zameer Manji.

Repository: aurora


This patch proposes that users installing custom modules do so via a custom main.

Specifying custom modules on the command line has proven troublesome for replacing the command
line args system with one where all arguments are injected and explicitly-defined.  It also
adds complexity to the scheduler application by unnecessarily punching holes at specific places.

If this proposal is agreeable, i will add a NEWS entry and document how one might implement
a custom main to add modules.  The tl;dr, however, is to invoke `SchedulerMain.publicMain(customModule)`

Diffs (updated)

  src/main/java/org/apache/aurora/scheduler/app/ d5f96543d1068bf30b9d173a247c2806faf35578

  src/main/java/org/apache/aurora/scheduler/app/ 0659c358479283756179c2cabebc8416730cc3e3

  src/main/java/org/apache/aurora/scheduler/http/api/security/ e32862034a1ad47dae8fff89cb04deb34ccd90e2

  src/main/java/org/apache/aurora/scheduler/http/api/security/ ccd9a20e8b18831458cba2d53e6b8b84fef06162

  src/test/java/org/apache/aurora/scheduler/app/ b2fb3c9dcba64f69a05894f506ba43766f74ddaa

  src/test/java/org/apache/aurora/scheduler/http/api/security/ 3e811a4f4d2c82892217ca1f950ac792303fbcf3

  src/test/java/org/apache/aurora/scheduler/http/api/security/ baaeb2390a909de1a92e4328d35a49f7b74c36cb



end-to-end tests, `./gradlew run`


Bill Farner

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