You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by mi...@apache.org on 2018/08/27 21:09:05 UTC
impala git commit: update Flask to latest (1.0.2)
Repository: impala
Updated Branches:
refs/heads/master c285ab979 -> c6ce735d1
update Flask to latest (1.0.2)
Similar to Fabric and Paramiko, make Flask part of extended test
requirements and upgrade it to its latest.
Testing:
- Impala builds, can do a full data load.
- Change works in my local environment.
Change-Id: Ibfc01d562e4d7fe48443d15074bd4c7d0176d2a0
Reviewed-on: http://gerrit.cloudera.org:8080/11335
Reviewed-by: Philip Zeyliger <ph...@cloudera.com>
Tested-by: Impala Public Jenkins <im...@cloudera.com>
Project: http://git-wip-us.apache.org/repos/asf/impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/c6ce735d
Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/c6ce735d
Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/c6ce735d
Branch: refs/heads/master
Commit: c6ce735d1b2e050f7e363d93267fc4b84e71f2f7
Parents: c285ab9
Author: Michael Brown <mi...@cloudera.com>
Authored: Mon Aug 27 08:33:57 2018 -0700
Committer: Impala Public Jenkins <im...@cloudera.com>
Committed: Mon Aug 27 21:01:51 2018 +0000
----------------------------------------------------------------------
infra/python/deps/extended-test-requirements.txt | 3 +++
infra/python/deps/requirements.txt | 5 -----
tests/comparison/leopard/front_end.py | 7 ++++++-
3 files changed, 9 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/impala/blob/c6ce735d/infra/python/deps/extended-test-requirements.txt
----------------------------------------------------------------------
diff --git a/infra/python/deps/extended-test-requirements.txt b/infra/python/deps/extended-test-requirements.txt
index 4876ec8..3f70b8d 100644
--- a/infra/python/deps/extended-test-requirements.txt
+++ b/infra/python/deps/extended-test-requirements.txt
@@ -19,6 +19,9 @@
# The random query generator uses Fabric to interact with a remote host
# running Docker.
Fabric==1.14.0
+# The leopard framework, which runs a webui to view random query generator
+# reports, requires Flask.
+Flask==1.0.2
# Fabric depends on Paramiko. Additionally, the stress test uses
# Paramiko directly to keep a persistent SSH connection open to each
# Impalad host to run in-test monitoring.
http://git-wip-us.apache.org/repos/asf/impala/blob/c6ce735d/infra/python/deps/requirements.txt
----------------------------------------------------------------------
diff --git a/infra/python/deps/requirements.txt b/infra/python/deps/requirements.txt
index a92fdc6..fd07ca8 100644
--- a/infra/python/deps/requirements.txt
+++ b/infra/python/deps/requirements.txt
@@ -38,11 +38,6 @@ flake8 == 2.6.2
mccabe == 0.3.1
pycodestyle == 2.0.0
pyflakes == 1.2.3
-Flask == 0.10.1
- Jinja2 == 2.8
- MarkupSafe == 0.23
- Werkzeug == 0.11.3
- itsdangerous == 0.24
gcovr == 3.4
kazoo == 2.2.1
ordereddict == 1.1
http://git-wip-us.apache.org/repos/asf/impala/blob/c6ce735d/tests/comparison/leopard/front_end.py
----------------------------------------------------------------------
diff --git a/tests/comparison/leopard/front_end.py b/tests/comparison/leopard/front_end.py
index e5d5b22..9752f7b 100755
--- a/tests/comparison/leopard/front_end.py
+++ b/tests/comparison/leopard/front_end.py
@@ -23,7 +23,12 @@ import pickle
import stat
import time
from time import sleep
-from flask import Flask, render_template, request
+try:
+ from flask import Flask, render_template, request
+except ImportError as e:
+ raise Exception(
+ "Please run impala-pip install -r $IMPALA_HOME/infra/python/deps/extended-test-"
+ "requirements.txt:\n{0}".format(str(e)))
from schedule_item import ScheduleItem
from controller import PATH_TO_REPORTS, PATH_TO_SCHEDULE
from threading import Thread