aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <>
Subject Re: Review Request 29696: Fix bad call to print_out causing 'job inspect --raw' to fail.
Date Fri, 09 Jan 2015 18:31:17 GMT

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

(Updated Jan. 9, 2015, 6:31 p.m.)

Review request for Aurora, Kevin Sweeney and Zameer Manji.


Fixed import sort order.  Not sure why reviewbot is missing these things...

Bugs: AURORA-990

Repository: aurora


In the course of this, i took a stab at cleaning up the unit test to remove unnecessary mocking
and avoid writing to a temporary file.  The nice outcome of the mock removal is that the job
configuration in our code is identical to what would appear in a configuration file.

Diffs (updated)

  src/main/python/apache/aurora/client/cli/ 395819fdf24b7919b32be51060fb5b581c8e1514

  src/main/python/apache/aurora/client/cli/ 744d53405081e75b9aedcdcb7e4a823a2e0e743f

  src/test/python/apache/aurora/client/cli/ 1c07d912c95eafa292d5451f4d6c72e9a711dae0




Bill Farner

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