[ARVADOS] updated: 827f01731858dc21578f0bd35f01d9d58b72d2ef
Git user
git at public.curoverse.com
Tue Apr 19 10:19:11 EDT 2016
Summary of changes:
services/keep-web/handler.go | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
discards 049b7e0eb603e672f56248fcd500acaf03c474c8 (commit)
via 827f01731858dc21578f0bd35f01d9d58b72d2ef (commit)
This update added new revisions after undoing existing revisions. That is
to say, the old revision is not a strict subset of the new revision. This
situation occurs when you --force push a change and generate a repository
containing something like this:
* -- * -- B -- O -- O -- O (049b7e0eb603e672f56248fcd500acaf03c474c8)
\
N -- N -- N (827f01731858dc21578f0bd35f01d9d58b72d2ef)
When this happens we assume that you've already had alert emails for all
of the O revisions, and so we here report only the revisions in the N
branch from the common base, B.
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 827f01731858dc21578f0bd35f01d9d58b72d2ef
Author: Tom Clegg <tom at curoverse.com>
Date: Tue Apr 19 10:18:51 2016 -0400
9005: Workaround: Close idle connections aggressively.
Currently, the SDK code never reuses connections anyway, so it's best
to shut them down right away.
diff --git a/services/keep-web/handler.go b/services/keep-web/handler.go
index e1b2362..6f5f66a 100644
--- a/services/keep-web/handler.go
+++ b/services/keep-web/handler.go
@@ -320,6 +320,12 @@ func (h *handler) ServeHTTP(wOrig http.ResponseWriter, r *http.Request) {
statusCode, statusText = http.StatusInternalServerError, err.Error()
return
}
+ if kc.Client != nil && kc.Client.Transport != nil {
+ // Workaround for https://dev.arvados.org/issues/9005
+ if t, ok := kc.Client.Transport.(*http.Transport); ok {
+ defer t.CloseIdleConnections()
+ }
+ }
rdr, err := kc.CollectionFileReader(collection, filename)
if os.IsNotExist(err) {
statusCode = http.StatusNotFound
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list