Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22452 )
Change subject: IMPALA-13201: Remove Unused Parameter from Test retry Function ...................................................................... IMPALA-13201: Remove Unused Parameter from Test retry Function The custom cluster tests utilize the retry() function defined in retry.py. This function takes as input another function to do the assertions. This assertion function used to have a single boolean parameter indicating if the retry was on its last attempt. In actuality, this boolean was not used and thus caused flake8 failures. This change removes this unused parameter from the assertion function passed in to the retry function. Change-Id: I1bce9417b603faea7233c70bde3816beed45539e Reviewed-on: http://gerrit.cloudera.org:8080/22452 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M tests/common/custom_cluster_test_suite.py M tests/common/impala_test_suite.py M tests/custom_cluster/test_query_live.py M tests/custom_cluster/test_query_log.py M tests/util/retry.py 5 files changed, 10 insertions(+), 18 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/22452 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1bce9417b603faea7233c70bde3816beed45539e Gerrit-Change-Number: 22452 Gerrit-PatchSet: 5 Gerrit-Owner: Jason Fehr <[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]>
