impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Internal Jenkins (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4122: qgen: fix bitrotted cluster unit tests
Date Wed, 14 Sep 2016 00:24:06 GMT
Internal Jenkins has submitted this change and it was merged.

Change subject: IMPALA-4122: qgen: fix bitrotted cluster unit tests

IMPALA-4122: qgen: fix bitrotted cluster unit tests

There's a small set of pytest-style tests and associated conftest for
testing some of the cluster-related test infrastructure Python objects.
Going forward, I want unit tests for the query generator to be run as
part of patch acceptance (CI isn't necessary at this time).

This patch fixes a bitrotted test and moves the tests into the
tests-for-qgen directory. The moves were performed thusly:

 $ git mv tests/
 $ git mv tests/

Change-Id: I3e855e265ae245ebe3691d077284ac5761909e00
Reviewed-by: Michael Brown <>
Reviewed-by: Tim Armstrong <>
Tested-by: Internal Jenkins
D tests/comparison/
A tests/comparison/tests/
R tests/comparison/tests/
3 files changed, 66 insertions(+), 50 deletions(-)

  Michael Brown: Looks good to me, but someone else must approve
  Internal Jenkins: Verified
  Tim Armstrong: Looks good to me, approved

To view, visit
To unsubscribe, visit

Gerrit-MessageType: merged
Gerrit-Change-Id: I3e855e265ae245ebe3691d077284ac5761909e00
Gerrit-PatchSet: 6
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Michael Brown <>
Gerrit-Reviewer: David Knupp <>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Michael Brown <>
Gerrit-Reviewer: Tim Armstrong <>

View raw message