impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Knupp (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4510: Selectively filter args for metric verification tests
Date Fri, 18 Nov 2016 08:06:14 GMT
David Knupp has uploaded a new patch set (#2).

Change subject: IMPALA-4510: Selectively filter args for metric verification tests

IMPALA-4510: Selectively filter args for metric verification tests is a wrapper around impala-pytest. It abstracts away
the need to invoke separate individual runs for serial tests,
parallel tests, and metric verification tests.

Because it's possible for a user to specify certain test suites,
or even specific tests, on the command line when calling, it had been necessary to override the command line
args when it came time to run the metric verification tests --
otherwise those other tests/suites would be rerun. Before this
patch, we had simply been stripping away all command line args.

However, that blanket approach causes problems when running tests
against a remote cluster, because we need to retain those command
line args that pertain to the remote cluster.

This patch selectively prunes unwanted command line args for the
last metric verification test stage, keeping the ones that we
need, and also adds extensive documentation for explaining why we
have to go through this fairly odd and elaborate step.

Change-Id: I069172f44c1307d55f85779cdb01fecc0ba1799e
M tests/
1 file changed, 55 insertions(+), 11 deletions(-)

  git pull ssh:// refs/changes/35/5135/2
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I069172f44c1307d55f85779cdb01fecc0ba1799e
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: David Knupp <>

View raw message