Zach Amsden has posted comments on this change.

Change subject: IMPALA-5127: Add history_max option
......................................................................


Patch Set 5:

Also, ran shell tests just for completeness:

impala@impala-dev:~/Impala$ tests/run-tests.py shell
================================================= test session starts 
=================================================
platform linux2 -- Python 2.7.6, pytest-2.9.2, py-1.4.32, pluggy-0.3.1 -- 
/home/impala/Impala/bin/../infra/python/env/bin/python
cachedir: .cache
rootdir: /home/impala/Impala/tests, inifile: pytest.ini
plugins: random-0.2, xdist-1.15.0
collected 44 items

shell/test_shell_commandline.py::TestImpalaShell::test_refresh_on_connect PASSED
shell/test_shell_commandline.py::TestImpalaShell::test_kerberos_option PASSED
shell/test_shell_commandline.py::TestImpalaShell::test_queries_closed PASSED

etc..

-- 
To view, visit http://gerrit.cloudera.org:8080/6335
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaf65bbecb8fd7f1105aac62b6745d6125a603d7f
Gerrit-PatchSet: 5
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Zach Amsden <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Michael Brown <[email protected]>
Gerrit-Reviewer: Sailesh Mukil <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Zach Amsden <[email protected]>
Gerrit-HasComments: No

Reply via email to