----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55237/ -----------------------------------------------------------
(Updated Jan. 10, 2017, 11:21 p.m.) Review request for Aurora and Santhosh Kumar Shanmugham. Repository: aurora Description ------- WIP for idempotent requests. The changes are basically the following: 1. In the client we construct a thrift transport based on the requests library. A new `TRequestsTransportRetry` adds a http header called `Idempotency-Token`. The retry logic is moved here so that retries will use the same idempotency token. I use the retry library for exponential backoff 2. In the scheduler `TContentAwareServlet` is provided with a `RequestCache` which is nothing but a Guava cache which expires keys after an hour. Diffs ----- 3rdparty/python/requirements.txt df2db7ac1a1d62465731c057eeeef67000cb589a RELEASE-NOTES.md 90c4793b218f54ad92ab4da5049d5e92c3a104b3 src/main/java/org/apache/aurora/scheduler/http/api/ApiModule.java e46820952fb6028911bca924169ceade6a134bfc src/main/java/org/apache/aurora/scheduler/http/api/RequestCache.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/http/api/TContentAwareServlet.java d9df96896d83786596880105968dc7fcdc0168e5 src/main/python/apache/aurora/admin/BUILD b5d37f718a25d43d7ac07e30f789cf526bdfdcc2 src/main/python/apache/aurora/admin/admin.py 070c348d2ca5db1edecf832efd9aa5481bddaa4b src/main/python/apache/aurora/admin/aurora_admin.py fbebbab8c827b5695042d18770d850e31fc38122 src/main/python/apache/aurora/client/api/scheduler_client.py 9bbfece012e48e0b1752bbefd25c89e04d312cf6 src/main/python/apache/aurora/common/BUILD 0e4c51020946dc21953493bd43b944177c444c28 src/main/python/apache/aurora/common/transport.py f44d13afa06112ff7b03a68cd01e7d9ef9e550ea src/test/java/org/apache/aurora/scheduler/http/api/TContentAwareServletTests.java PRE-CREATION src/test/python/apache/aurora/client/api/test_scheduler_client.py f6018caa4f431e85a9e9ff203ac3d4b6c33f40ef src/test/python/apache/aurora/common/test_transport.py c3d35d278a68db603954ad1e3cf26c6359ae3926 Diff: https://reviews.apache.org/r/55237/diff/ Testing ------- Works locally. Not good for a review , this is just work in progress. Thanks, Karthik Anantha Padmanabhan