Mailing-List: contact dev-help@impala.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@impala.incubator.apache.org
Date: Fri, 3 Jun 2016 17:06:16 +0000
From: "David Knupp (Code Review)" <gerrit@cloudera.org>
To: impala-cr@cloudera.com, dev@impala.incubator.apache.org
Message-ID: <gerrit.1464973431000.Icdcb04a99c0907fc1ba56baa2497fafb33b0e34e@gerrit.cloudera.org>
Reply-To: dknupp@cloudera.com
Subject: =?UTF-8?Q?=5BImpala-CR=5D=28cdh5-trunk=29_IMPALA-3491=3A_Use_unique_database_fixture_in_test_shell_commandline=2Epy=2E=0A?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Content-Disposition: inline
User-Agent: Gerrit/2.12.2
archived-at: Fri, 03 Jun 2016 17:06:22 -0000

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>