Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/14629


Change subject: IMPALA-9100: Handle duplicate occurrences of flags for 
tests/run-tests.py
......................................................................

IMPALA-9100: Handle duplicate occurrences of flags for tests/run-tests.py

If someone passes --skip-stress multiple times to tests/run-tests.py,
it currently only removes one of the occurrences from the arguments
and allows the other one to pass through to pytest. This causes pytest
to immediately error out. This behavior is seen on the docker-based
tests, because test-with-docker.py specifies --skip-stress and
bin/run-all-tests.sh adds another --skip-stress for core runs.

This changes tests/run-tests.py to handle multiple occurrences of
--skip-stress, --skip-parallel, and --skip-serial.

Testing:
 - Tested manually with duplicate skip flags.

Change-Id: I60dc9a898f69804e2a53c05b5dfab2f948a22097
---
M tests/run-tests.py
1 file changed, 14 insertions(+), 9 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I60dc9a898f69804e2a53c05b5dfab2f948a22097
Gerrit-Change-Number: 14629
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>

Reply via email to