From notifications-return-123540-archive-asf-public=cust-asf.ponee.io@asterixdb.apache.org Sun Oct 28 00:34:34 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 6F894180649 for ; Sun, 28 Oct 2018 00:34:34 +0200 (CEST) Received: (qmail 71924 invoked by uid 500); 27 Oct 2018 22:34:33 -0000 Mailing-List: contact notifications-help@asterixdb.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@asterixdb.apache.org Delivered-To: mailing list notifications@asterixdb.apache.org Received: (qmail 71915 invoked by uid 99); 27 Oct 2018 22:34:33 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 27 Oct 2018 22:34:33 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id E520418CD44 for ; Sat, 27 Oct 2018 22:34:32 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.919 X-Spam-Level: X-Spam-Status: No, score=0.919 tagged_above=-999 required=6.31 tests=[RCVD_IN_DNSWL_NONE=-0.0001, SPF_FAIL=0.919] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id vSIW7yfdqATy for ; Sat, 27 Oct 2018 22:34:32 +0000 (UTC) Received: from vitalstatistix.ics.uci.edu (vitalstatistix.ics.uci.edu [128.195.52.38]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id 5741A5F3B7 for ; Sat, 27 Oct 2018 22:34:31 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by vitalstatistix.ics.uci.edu (Postfix) with ESMTP id ADC661000AE; Sat, 27 Oct 2018 15:34:30 -0700 (PDT) Date: Sat, 27 Oct 2018 15:34:30 -0700 From: "Murtadha Hubail (Code Review)" To: Jenkins CC: Till Westmann , Michael Blow Reply-To: mhubail@apache.org X-Gerrit-MessageType: newpatchset Subject: Change in asterixdb[master]: [NO ISSUE][OTH] Do Not Close Client Connection After Failure X-Gerrit-Change-Id: Id0fce2c860eec97f3d368ee42f25dbdfc9dc0ff9 X-Gerrit-ChangeURL: X-Gerrit-Commit: 2ca414ce92595b621226d4bba178d76b1d354cb6 In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Content-Disposition: inline User-Agent: Gerrit/2.12.7 Message-Id: <20181027223430.ADC661000AE@vitalstatistix.ics.uci.edu> Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/3006 to look at the new patch set (#2). Change subject: [NO ISSUE][OTH] Do Not Close Client Connection After Failure ...................................................................... [NO ISSUE][OTH] Do Not Close Client Connection After Failure - user model changes: no - storage format changes: no - interface changes: no Details: - Currently, after sending some failure responses (e.g. after servlet not found), the client connection is closed even if the connection was supposed to be kept alive. This change ensures that we do not close the client connection -- if keep alive is requested -- which allows the client to submit another request using the same connection. - Ensure a full http response is sent to the client in case of a failure and not only the response header. Change-Id: Id0fce2c860eec97f3d368ee42f25dbdfc9dc0ff9 --- M hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServerHandler.java 1 file changed, 10 insertions(+), 7 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/06/3006/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/3006 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id0fce2c860eec97f3d368ee42f25dbdfc9dc0ff9 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann