[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