Riza Suminto has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23076 )

Change subject: IMPALA-13917 (part 1): Remove Beeswax from protocol dimension
......................................................................

IMPALA-13917 (part 1): Remove Beeswax from protocol dimension

Beeswax protocol has been due for deprecation in a long time. This patch
remove BEESWAX from create_client_protocol_dimension(). This will limit
protocol dimension to [HS2, HS2_HTTP] by default. It is still possible
to include BEESWAX again for testing if DEFAULT_TEST_PROTOCOL env var is
set to 'beeswax', such as:

DEFAULT_TEST_PROTOCOL=beeswax impala-py.test custom_cluster/test_ipv6.py

This patch does not disable beeswax server yet. Some tests that
specifically test against beeswax protocol, such as test_beeswax.py,
will continue to work. ImpalaTestSuite.beeswax_client also remain
unchanged.

Testing:
Run following command and confirm that beeswax protocol is skipped.

impala-py.test --collect-only --exploration=exhaustive \
  custom_cluster/test_ipv6.py

Change-Id: I3cff79f59305b5d44944804ed1f1b92838575495
Reviewed-on: http://gerrit.cloudera.org:8080/23076
Reviewed-by: Jason Fehr <[email protected]>
Tested-by: Riza Suminto <[email protected]>
---
M tests/common/test_dimensions.py
1 file changed, 7 insertions(+), 9 deletions(-)

Approvals:
  Jason Fehr: Looks good to me, approved
  Riza Suminto: Verified

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3cff79f59305b5d44944804ed1f1b92838575495
Gerrit-Change-Number: 23076
Gerrit-PatchSet: 5
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Pranav Lodha <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to