Mailing-List: contact reviews-help@impala.apache.org; run by ezmlm
Precedence: bulk
Message-Id: <201712151153.vBFBrqPa000384@ip-10-146-233-104.ec2.internal>
Date: Fri, 15 Dec 2017 11:53:52 +0000
From: "Gabor Kaszab (Code Review)" <gerrit@cloudera.org>
To: Laszlo Gaal <laszlo.gaal@cloudera.com>,
        Zoltan Borok-Nagy <boroknagyz@cloudera.com>,
        David Knupp <dknupp@cloudera.com>, Attila Jeges <attilaj@cloudera.com>,
        Tim Armstrong <tarmstrong@cloudera.com>,
        Csaba Ringhofer <csringhofer@cloudera.com>, impala-cr@cloudera.com,
        reviews@impala.incubator.apache.org
Subject: =?UTF-8?Q?=5BImpala-ASF-CR=5D_IMPALA-6318=3A_Adjustment_for_hanging_query_cancellation_test=0A?=
In-Reply-To: <gerrit.1513327949000.I082c83b91b6d0c527de92c7992f0dc9d1b290433@gerrit.cloudera.org>
References: <gerrit.1513327949000.I082c83b91b6d0c527de92c7992f0dc9d1b290433@gerrit.cloudera.org>
Reply-To: gaborkaszab@cloudera.com, impala-cr@cloudera.com,
        dknupp@cloudera.com, boroknagyz@cloudera.com, marcelk@gmail.com,
        tarmstrong@cloudera.com, laszlo.gaal@cloudera.com,
        csringhofer@cloudera.com, attilaj@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="+20Vk7zdVuw="; charset=UTF-8
archived-at: Fri, 15 Dec 2017 11:53:57 -0000

--+20Vk7zdVuw=
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hello Laszlo Gaal, Zoltan Borok-Nagy, David Knupp, Attila Jeges, Tim Armstr=
ong, Csaba Ringhofer, 

I'd like you to reexamine a change=2E Please visit
=

    http://gerrit=2Ecloudera=2Eorg:8080/8852

to look at the new patch set=
 (#3)=2E

Change subject: IMPALA-6318: Adjustment for hanging query cancell=
ation test
=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

I=
MPALA-6318: Adjustment for hanging query cancellation test

Apparently test=
_query_cancellation_during_fetch hangs occasionally
in Jenkins builds=2E Th=
e Impala debug page shows the query being
cancelled, however, on the host t=
he ImpalaShell process related to
that query is still running=2E

Since I h=
ad no luck in reproducing the issue locally I only have a
theory what might=
 be going on here: The query is cancelled
successfully on Impala backend an=
d when the test tries to get the
stdout and stderr from the ImpalaShell it =
gets stuck=2E It might be
the case that ImpalaShell process fetching the qu=
ery results holds
the stdout, so as a workaround there is a new optional pa=
rameter to
util=2EImpalaShell to omit the stdout because this test wouldn't=
 use
it anyway=2E

Change-Id: I082c83b91b6d0c527de92c7992f0dc9d1b290433
---=

M tests/shell/test_shell_commandline=2Epy
M tests/shell/util=2Epy
2 files =
changed, 8 insertions(+), 6 deletions(-)


  git pull ssh://gerrit=2Ecloude=
ra=2Eorg:29418/Impala-ASF refs/changes/52/8852/3
-- 
To view, visit http://=
gerrit=2Ecloudera=2Eorg:8080/8852
To unsubscribe, visit http://gerrit=2Eclo=
udera=2Eorg:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master=

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I082c83b91b6d0c527de92c7=
992f0dc9d1b290433
Gerrit-Change-Number: 8852
Gerrit-PatchSet: 3
Gerrit-Owne=
r: Gabor Kaszab <gaborkaszab@cloudera=2Ecom>
Gerrit-Reviewer: Attila Jeges =
<attilaj@cloudera=2Ecom>
Gerrit-Reviewer: Csaba Ringhofer <csringhofer@clou=
dera=2Ecom>
Gerrit-Reviewer: David Knupp <dknupp@cloudera=2Ecom>
Gerrit-Rev=
iewer: Laszlo Gaal <laszlo=2Egaal@cloudera=2Ecom>
Gerrit-Reviewer: Tim Arms=
trong <tarmstrong@cloudera=2Ecom>
Gerrit-Reviewer: Zoltan Borok-Nagy <borok=
nagyz@cloudera=2Ecom>

--+20Vk7zdVuw=--