[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