impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Knupp (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3491: Use unique database fixture in test shell commandline.py.
Date Fri, 03 Jun 2016 17:06:16 GMT
David Knupp has uploaded a new change for review.

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

Change subject: IMPALA-3491: Use unique_database fixture in test_shell_commandline.py.
......................................................................

IMPALA-3491: Use unique_database fixture in test_shell_commandline.py.

Before this change, a single test database was created for the entire suite,
and each test was marked to run serially. With the addition of a test fixture
in tests/conftest.py to create a unique database per each individual method,
it's possible now to run the tests in parallel. (The tables required by
individual tests are created via local test fixtures.)

As such, any methods which had been responsible for setting up the test
database were removed. Pytest markers for running tests serially were also
removed, except in cases where interactions from running concurrency would
affect other tests.

Additonal minor changes were made to improve PEP-8 compliance.

The non-serial tests were run in a loop ten times to confirm that there weren't
any unexpected failures.

Change-Id: Icdcb04a99c0907fc1ba56baa2497fafb33b0e34e
---
M tests/shell/test_shell_commandline.py
1 file changed, 93 insertions(+), 90 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icdcb04a99c0907fc1ba56baa2497fafb33b0e34e
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: David Knupp <dknupp@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Michael Brown <mikeb@cloudera.com>

Mime
View raw message