Hello Jason Fehr, Michael Smith, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#2).

Change subject: IMPALA-14667: Remove reference to pytest.config
......................................................................

IMPALA-14667: Remove reference to pytest.config

pytest.config global variable has been removed since pytest 5.0.
https://docs.pytest.org/en/stable/deprecations.html#pytest-config-global

This patch remove direct reference to pytest.config. Instead, it hook
pytest config object to ImpalaTestClusterProperties singleton at
pytest_configure() method at conftest.py.

Added pytest_config() classmethod in BaseTestSuite as a shortcut to
access ImpalaTestClusterProperties.get_instance().pytest_config().

Testing:
- Pass core tests.

Change-Id: Ieb09a9c8c1f474c355188b46a0ea913da6c2a38b
---
M tests/catalog_service/test_catalog_service_client.py
M tests/common/base_test_suite.py
M tests/common/custom_cluster_test_suite.py
M tests/common/environ.py
M tests/common/impala_test_suite.py
M tests/common/kudu_test_suite.py
M tests/common/skip.py
M tests/common/test_dimensions.py
M tests/conftest.py
M tests/custom_cluster/test_event_processing_perf.py
M tests/custom_cluster/test_insert_behaviour.py
M tests/custom_cluster/test_kudu.py
M tests/custom_cluster/test_saml2_sso.py
M tests/metadata/test_partition_metadata.py
M tests/query_test/test_kudu.py
M tests/run-tests.py
M tests/shell/util.py
17 files changed, 165 insertions(+), 113 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/40/23840/2
--
To view, visit http://gerrit.cloudera.org:8080/23840
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ieb09a9c8c1f474c355188b46a0ea913da6c2a38b
Gerrit-Change-Number: 23840
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to