Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id F186E200D61 for ; Tue, 5 Dec 2017 06:14:12 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id EBEC6160C05; Tue, 5 Dec 2017 05:14:12 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 3A732160BF9 for ; Tue, 5 Dec 2017 06:14:12 +0100 (CET) Received: (qmail 97234 invoked by uid 500); 5 Dec 2017 05:14:11 -0000 Mailing-List: contact reviews-help@impala.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list reviews@impala.apache.org Received: (qmail 97223 invoked by uid 99); 5 Dec 2017 05:14:11 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Dec 2017 05:14:11 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id 3C3B3C3F11 for ; Tue, 5 Dec 2017 05:14:10 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 2.362 X-Spam-Level: ** X-Spam-Status: No, score=2.362 tagged_above=-999 required=6.31 tests=[HTML_MESSAGE=2, RDNS_DYNAMIC=0.363, SPF_PASS=-0.001] autolearn=disabled Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id CxL8kNvXaDDn for ; Tue, 5 Dec 2017 05:14:09 +0000 (UTC) Received: from ip-10-146-233-104.ec2.internal (ec2-75-101-130-251.compute-1.amazonaws.com [75.101.130.251]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTPS id 3C1705F393 for ; Tue, 5 Dec 2017 05:14:09 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by ip-10-146-233-104.ec2.internal (8.14.4/8.14.4) with ESMTP id vB55E8mB024787; Tue, 5 Dec 2017 05:14:08 GMT Message-Id: <201712050514.vB55E8mB024787@ip-10-146-233-104.ec2.internal> X-Gerrit-PatchSet: 6 Date: Tue, 5 Dec 2017 05:14:08 +0000 From: "Sailesh Mukil (Code Review)" To: impala-cr@cloudera.com, reviews@impala.incubator.apache.org CC: Dan Hecht , Michael Ho , Lars Volker X-Gerrit-MessageType: comment Subject: =?UTF-8?Q?=5BImpala-ASF-CR=5D_IMPALA-4671=3A_=28part-2=29_Replace_kudu=3A=3AServicePool_with_one_that_uses_Impala_threads=0A?= X-Gerrit-Change-Id: Ia3e0fa9ae22c4803b0cea736eb35e333e2671448 X-Gerrit-Change-Number: 8472 X-Gerrit-ChangeURL: X-Gerrit-Commit: 3993c05aad02fc58f71fb727cc0cce2a2ee08243 In-Reply-To: References: X-Gerrit-Comment-Date: Tue, 5 Dec 2017 05:14:08 +0000 Reply-To: sailesh@cloudera.com, impala-cr@cloudera.com, lv@cloudera.com, marcelk@gmail.com, dhecht@cloudera.com, kwho@cloudera.com, reviews@impala.incubator.apache.org MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Disposition: inline User-Agent: Gerrit/2.14.2 Content-Type: multipart/alternative; boundary="hku4H9rqB/g="; charset=UTF-8 archived-at: Tue, 05 Dec 2017 05:14:13 -0000 --hku4H9rqB/g= Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Sailesh Mukil has posted comments on this change=2E ( http://gerrit=2Ecloud= era=2Eorg:8080/8472 ) Change subject: IMPALA-4671: (part-2) Replace kudu::= ServicePool with one that uses Impala threads =2E=2E=2E=2E=2E=2E=2E=2E=2E= =2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E= =2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E= =2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E Patch Set 6: Code-Review+2 (1 comment)= Rebase, carry +2=2E http://gerrit=2Ecloudera=2Eorg:8080/#/c/8472/5/be/sr= c/rpc/impala-service-pool=2Eh File be/src/rpc/impala-service-pool=2Eh: htt= p://gerrit=2Ecloudera=2Eorg:8080/#/c/8472/5/be/src/rpc/impala-service-pool= =2Eh@76 PS5, Line 76: against c > what resources? Let's just remove that w= ord since we've been trying to be Done -- To view, visit http://gerrit= =2Ecloudera=2Eorg:8080/8472 To unsubscribe, visit http://gerrit=2Ecloudera= =2Eorg:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerr= it-MessageType: comment Gerrit-Change-Id: Ia3e0fa9ae22c4803b0cea736eb35e333= e2671448 Gerrit-Change-Number: 8472 Gerrit-PatchSet: 6 Gerrit-Owner: Sailes= h Mukil Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: = Michael Ho Gerrit-Reviewer: Sailesh Mukil Gerrit-Comment-Date: Tue, 05 Dec 2017 05:14:08 +0000 Gerrit-H= asComments: Yes --hku4H9rqB/g=--