> On Jan. 4, 2018, 7 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/configuration/executor/ExecutorModule.java > > Lines 73 (patched) > > <https://reviews.apache.org/r/64934/diff/1/?file=1930104#file1930104line73> > > > > This bug affects all `@Parameter(..) List<>` fields. Can you apply to > > them as well? > > > > This should identify all the List parameters: > > ``` > > $ grep ImmutableList.of > > src/test/java/org/apache/aurora/scheduler/config/CommandLineTest.java > > ```
Sure, working on this, hope to have it done by EOD today. - Renan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64934/#review194826 ----------------------------------------------------------- On Jan. 3, 2018, 3:29 p.m., Renan DelValle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64934/ > ----------------------------------------------------------- > > (Updated Jan. 3, 2018, 3:29 p.m.) > > > Review request for Aurora, Stephan Erb and Bill Farner. > > > Repository: aurora > > > Description > ------- > > Fixes the issue that caused the voting to fail for the 0.19.0 Aurora packages. > > Fix cribbed from: https://github.com/cbeust/jcommander/pull/422 > > Implemented as a custom converter as suggested here: > https://reviews.apache.org/r/64824/ > > > Diffs > ----- > > > src/main/java/org/apache/aurora/scheduler/config/converters/StringListConverter.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/configuration/executor/ExecutorModule.java > 76ce39e837ea9dce81fdc66018611dfd11612388 > > > Diff: https://reviews.apache.org/r/64934/diff/1/ > > > Testing > ------- > > End to end. > Style check. > Integration. > Ran vagrant box, added `-thermos_executor_resources=""` created and killed a > job. > > > Thanks, > > Renan DelValle > >
