Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21036 )
Change subject: IMPALA-12814: Allow use of row_regex for VERIFY_IS_NOT_IN ...................................................................... IMPALA-12814: Allow use of row_regex for VERIFY_IS_NOT_IN This adds support for using "row_regex:" entries in the VERIFY_IS_NOT_IN sections. There are existing tests that already use this formula, but it doesn't actually match using the regex. This fixes test_result_verifier.py's verify_query_result_is_not_in() to detect the regexes and use regex logic for the check. Testing: - Hand tested by adding a "row_regex:.*" to a VERIFY_IS_NOT_IN section and verifying it produces an error. - Hand tested by adding a "row_regex:match_nothing.*" to make sure nothing matched it. Change-Id: Id856ae7de9727adeaec9487c7bd0c2b81c8e8b37 Reviewed-on: http://gerrit.cloudera.org:8080/21036 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Michael Smith <[email protected]> --- M tests/common/test_result_verifier.py 1 file changed, 20 insertions(+), 4 deletions(-) Approvals: Impala Public Jenkins: Verified Michael Smith: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/21036 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id856ae7de9727adeaec9487c7bd0c2b81c8e8b37 Gerrit-Change-Number: 21036 Gerrit-PatchSet: 3 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
