> On Jan. 23, 2015, 12:47 p.m., Maxim Khutornenko wrote:
> > I am not convinced there is enough value in this diff to risk possible 
> > regression. Besides, the majority of what this diff touches will die out 
> > along with the client updater.
> 
> Zameer Manji wrote:
>     Is there an ETA for the destruction of the client updater?
> 
> Maxim Khutornenko wrote:
>     Any time we feel ready to drop "beta" from scheduler updater.

I'm willing to drop this diff, if you are willing to start the conversation on 
when we can drop 'beta' from the scheduler updater.


- Zameer


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


On Jan. 22, 2015, 7:32 p.m., Zameer Manji wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30207/
> -----------------------------------------------------------
> 
> (Updated Jan. 22, 2015, 7:32 p.m.)
> 
> 
> Review request for Aurora, Maxim Khutornenko and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> The AuroraCommandContext class is used in multiple commands and contains 
> common code for all of them. However some portions are only used by one 
> command. This patch takes some of those portions and moves them to the 
> command that requires that functionality.
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/aurora/client/cli/context.py 
> 51c7d24dca664e476e62f1864d095416dfab70e4 
>   src/main/python/apache/aurora/client/cli/jobs.py 
> 8f349c09637c16e2499e85f2dc96eb7ccffd0aaf 
>   src/main/python/apache/aurora/client/cli/update.py PRE-CREATION 
>   src/test/python/apache/aurora/client/cli/test_supdate.py PRE-CREATION 
>   src/test/python/apache/aurora/client/cli/test_update.py 
> 8b7d11202b35deb09a0000248cfe0a96458fb70c 
> 
> Diff: https://reviews.apache.org/r/30207/diff/
> 
> 
> Testing
> -------
> 
> ./pants test.pytest --no-fast src/test/python/apache/aurora/client::
> 
> 
> Thanks,
> 
> Zameer Manji
> 
>

Reply via email to