> On July 22, 2014, 12:54 a.m., Bill Farner wrote:
> > Do you think this should be obviated by https://reviews.apache.org/r/23741/ 
> > ?
> 
> Misho Krastev wrote:
>     This looks like elegant (declarative) solution to export REST variant of 
> the native thrift interface. I am not familiar with the Aurora scheduler code 
> so I can not read very well the diffs. From the review request description 
> and what I understand from the diffs, I would vote for that (your) solution.
>     
>     However, I don't think this REST service can be replaced by the other 
> one. The reason is the REST service that I submitted for review is an 
> interface to the capabilities provided by the Aurora client, and this client 
> is functionality-rich. It adds more features on top of those offered by the 
> scheduler. In particular, what we needed was the ability to do rolling 
> upgrades where instances (shards) are upgraded in batches and not en bloc. 
> There are other examples but this is the one that is of great importance to 
> us.

Ah, yes - good point.  We're actually embarking on pulling update orchestration 
into the scheduler [1] this week, after which we will remove the updating logic 
from the client.


[1] 
http://mail-archives.apache.org/mod_mbox/incubator-aurora-dev/201407.mbox/browser


- Bill


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


On July 21, 2014, 10:58 p.m., Misho Krastev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23666/
> -----------------------------------------------------------
> 
> (Updated July 21, 2014, 10:58 p.m.)
> 
> 
> Review request for Aurora.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> REST service for aurora client commands, built using Tornado Web framework
> 
> This is a draft implementation of a REST server exposing APIs to call Aurora 
> client commands. It is a wrapper around the aurora command-line client and 
> aurora client code. It implements some but not all of the commands available 
> with the aurora command-line client. Comments on the code as well suggestions 
> for improvements and enhancements are requested.
> 
> This work is a eBay project that we would like to contribute to the Aurora 
> project.
> 
> Note: development is still ongoing to add unittests and to resolve the issues 
> discovered with the stress tests.
> Note(2): GitHub repo -- https://github.com/misho-kr/incubator-aurora
> 
> 
> Diffs
> -----
> 
>   3rdparty/python/BUILD 641d610deb58fd102ec584e3deaa388b51a6f221 
>   docs/client-rest-api.md PRE-CREATION 
>   src/main/python/apache/aurora/client/bin/BUILD 
> 43d747956df0611b0880f64df9955d5f5806901c 
>   src/main/python/apache/aurora/client/bin/aurora_rest.py PRE-CREATION 
>   src/main/python/apache/aurora/client/rest/BUILD PRE-CREATION 
>   src/main/python/apache/aurora/client/rest/application.py PRE-CREATION 
>   src/main/python/apache/aurora/client/rest/application_async.py PRE-CREATION 
>   src/main/python/apache/aurora/client/rest/coroutine_executor.py 
> PRE-CREATION 
>   src/main/python/apache/aurora/client/rest/external_executor.py PRE-CREATION 
>   src/main/python/apache/aurora/client/rest/internal_executor.py PRE-CREATION 
>   src/main/python/apache/aurora/client/rest/mp_executor.py PRE-CREATION 
>   src/main/python/apache/aurora/client/rest/mt_executor.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/23666/diff/
> 
> 
> Testing
> -------
> 
> Stress testing with Locust to verify the REST service can handle multiple 
> requests simultaneously. Not all of them passed, particularly the ones with 
> the internal executor that calls Aurora client api.
> 
> 
> Thanks,
> 
> Misho Krastev
> 
>

Reply via email to