[ARVADOS] updated: 3d4717f6f48957d58d46fa0ecb4e65e3f49b1858
git at public.curoverse.com
git at public.curoverse.com
Tue Oct 20 11:23:40 EDT 2015
Summary of changes:
.../_getting_started_popup.html.erb | 2 +-
sdk/go/keepclient/support.go | 5 +-
sdk/python/arvados/keep.py | 33 +++-
sdk/python/tests/test_keep_client.py | 193 +++++++++++++--------
4 files changed, 157 insertions(+), 76 deletions(-)
via 3d4717f6f48957d58d46fa0ecb4e65e3f49b1858 (commit)
via 66dd4459251971acceae41622e2e1221892a8872 (commit)
via d379c467be58c66b2f1e7acafc97634b269a1542 (commit)
via 6de9152319167bb32602501e02e18a89e333336d (commit)
via f2691496d53aa6367d4b777b67881b05b50ed85f (commit)
via 25af384efacda9f3c552eebfee42e14b785e9a16 (commit)
from 17568511436d6721de01870fead8995796243ee8 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit 3d4717f6f48957d58d46fa0ecb4e65e3f49b1858
Author: radhika <radhika at curoverse.com>
Date: Tue Oct 20 11:21:34 2015 -0400
7546: also retry when status code is 0, which is the case when a closed connection was used.
diff --git a/sdk/go/keepclient/support.go b/sdk/go/keepclient/support.go
index 47d3e3a..3657260 100644
--- a/sdk/go/keepclient/support.go
+++ b/sdk/go/keepclient/support.go
@@ -235,7 +235,7 @@ func (this KeepClient) putReplicas(
// Take the hash of locator and timestamp in order to identify this
// specific transaction in log statements.
- requestId := fmt.Sprintf("%x", md5.Sum([]byte(locator+time.Now().String())))[0:8]
+ requestId := fmt.Sprintf("%x", md5.Sum([]byte(hash+time.Now().String())))[0:8]
// Calculate the ordering for uploading to servers
sv := NewRootSorter(this.WritableLocalRoots(), hash).GetSortedRoots()
@@ -304,7 +304,8 @@ func (this KeepClient) putReplicas(
// good news!
remaining_replicas -= status.replicas_stored
locator = status.response
- } else if status.statusCode == 408 || status.statusCode == 429 || status.statusCode >= 500 {
+ } else if status.statusCode == 0 || status.statusCode == 408 || status.statusCode == 429 ||
+ (status.statusCode >= 500 && status.statusCode != 503) {
// Timeout, too many requests, or other server side failure
retryServers = append(retryServers, status.url[0:strings.LastIndex(status.url, "/")])
}
commit 66dd4459251971acceae41622e2e1221892a8872
Merge: 1756851 d379c46
Author: radhika <radhika at curoverse.com>
Date: Tue Oct 20 11:02:19 2015 -0400
Merge branch 'master' into 7546-put-retry
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list