I think making a reasonable effort to make it harder to accidentally do
this is worthwhile - there's no harm in it, and it might just help.
On Thu, Mar 13, 2014 at 10:07 AM, Kevin Sweeney <kevi...@apache.org> wrote:
> This is an automatically generated e-mail. To reply, visit:
> On March 12th, 2014, 3:25 p.m. PDT, *Mark Chu-Carroll* wrote:
> revision 1)
> def show_job_pretty(job):
> @app.command_option('--force', default=False, action='store_true',
> This is deliberate: the "kill" command doesn't have a "force" option. So
> this ensures that there's more than a search-and-replace killall for kill:
> you need to deliberately use the killall command, and specify the force
> The require --shards is also in this change - see the change above in kill.
> On March 12th, 2014, 3:29 p.m. PDT, *Maxim Khutornenko* wrote:
> Missed the --shards part, thanks.
> If someone is going to search-and-replace kill with killall without wanting
> to do that I'm not sure we can stop them from shooting themselves in the
> foot. Up to you but I'm in favor of dropping this required option.
> - Kevin
> On March 12th, 2014, 3:11 p.m. PDT, Mark Chu-Carroll wrote:
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> By Mark Chu-Carroll.
> *Updated March 12, 2014, 3:11 p.m.*
> *Bugs: * aurora-260 <https://issues.apache.org/jira/browse/aurora-260>
> *Repository: * aurora
> Add killall.
> - the kill command now requires a shards parameter.
> - the new killall command only works when run with "--force".
> - killall generates a scary warning message, and pauses to give
> the user a chance to abort.
> Modified the existing kill command's test suite, adding new tests of the new
> functionality. All pass.
> [sun-wukong incubator-aurora (killall)]$ ./pants
> Build operating on targets:
> ============================= test session starts
> platform darwin -- Python 2.6.8 -- py-1.4.20 -- pytest-2.5.2
> collected 26 items
> src/test/python/apache/aurora/client/commands/test_cancel_update.py ..
> src/test/python/apache/aurora/client/commands/test_create.py ......
> src/test/python/apache/aurora/client/commands/test_diff.py ...
> src/test/python/apache/aurora/client/commands/test_kill.py .....
> src/test/python/apache/aurora/client/commands/test_listjobs.py ..
> src/test/python/apache/aurora/client/commands/test_restart.py ...
> src/test/python/apache/aurora/client/commands/test_status.py ..
> src/test/python/apache/aurora/client/commands/test_update.py ...
> ========================= 26 passed in 11.34 seconds
> ..... SUCCESS
> - src/main/python/apache/aurora/client/commands/core.py
> - src/test/python/apache/aurora/client/commands/test_kill.py
> View Diff <https://reviews.apache.org/r/19159/diff/>