impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Brown (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-4427: leopard: make DOCKER IMAGE NAME required
Date Thu, 03 Nov 2016 19:28:38 GMT
Michael Brown has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/4936

Change subject: IMPALA-4427: leopard: make DOCKER_IMAGE_NAME required
......................................................................

IMPALA-4427: leopard: make DOCKER_IMAGE_NAME required

This patch now requires users of the Leopard framwork to supply a
DOCKER_IMAGE_NAME. The cloudera/impala image isn't being maintained, and
until Apache Impala (incubating) decides to publish its own image, no
default image name is possible. It is still possible to run the Leopard
framework against a homegrown Docker image that has Apache Impala
(incubating) and PostgreSQL installed: simply build such an image and
export the DOCKER_IMAGE_NAME environment variable before running the
controller.

While here, fix some flake8 non-whitespace problems.

Testing: short Leopard controller / query generator run.

Change-Id: Ic1cb96cb5c9a894f40e0892f3bdd3f3d0158e887
---
M tests/comparison/leopard/controller.py
M tests/comparison/leopard/impala_docker_env.py
2 files changed, 3 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/36/4936/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic1cb96cb5c9a894f40e0892f3bdd3f3d0158e887
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Michael Brown <mikeb@cloudera.com>

Mime
View raw message