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 <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> 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 <[email protected]> Authored: Mon Aug 27 08:33:57 2018 -0700 Committer: Impala Public Jenkins <[email protected]> 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
