[Impala-ASF-CR] IMPALA-6201: Fix test basic filters on ASAN
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8646 ) Change subject: IMPALA-6201: Fix test_basic_filters on ASAN .. IMPALA-6201: Fix test_basic_filters on ASAN TestRuntimeFilters.test_basic_filters is flaky on ASAN as sometimes the runtime filters aren't recieved within the specified RUNTIME_FILTER_WAIT_TIME_MS. This patch increases the timeout for ASAN builds. Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Reviewed-on: http://gerrit.cloudera.org:8080/8646 Reviewed-by: Sailesh MukilReviewed-by: Alex Behm Tested-by: Impala Public Jenkins --- M testdata/workloads/functional-query/queries/QueryTest/runtime_filters.test M tests/query_test/test_runtime_filters.py 2 files changed, 21 insertions(+), 20 deletions(-) Approvals: Sailesh Mukil: Looks good to me, but someone else must approve Alex Behm: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Gerrit-Change-Number: 8646 Gerrit-PatchSet: 2 Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Reviewer: Tim Armstrong
[Impala-ASF-CR] IMPALA-6201: Fix test basic filters on ASAN
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8646 ) Change subject: IMPALA-6201: Fix test_basic_filters on ASAN .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Gerrit-Change-Number: 8646 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Tue, 28 Nov 2017 03:01:38 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6201: Fix test basic filters on ASAN
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8646 ) Change subject: IMPALA-6201: Fix test_basic_filters on ASAN .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1530/ -- To view, visit http://gerrit.cloudera.org:8080/8646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Gerrit-Change-Number: 8646 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Mon, 27 Nov 2017 23:28:20 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6201: Fix test basic filters on ASAN
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8646 ) Change subject: IMPALA-6201: Fix test_basic_filters on ASAN .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8646/1/tests/query_test/test_runtime_filters.py File tests/query_test/test_runtime_filters.py: http://gerrit.cloudera.org:8080/#/c/8646/1/tests/query_test/test_runtime_filters.py@27 PS1, Line 27: WAIT_TIME_MS = specific_build_type_timeout(6, slow_build_timeout=10) > I just pushed out a fix with some info: https://gerrit.cloudera.org/#/c/865 No, I hadn't seen that. Good catch. -- To view, visit http://gerrit.cloudera.org:8080/8646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Gerrit-Change-Number: 8646 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Mon, 27 Nov 2017 23:28:00 + Gerrit-HasComments: Yes
[Impala-ASF-CR] IMPALA-6201: Fix test basic filters on ASAN
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8646 ) Change subject: IMPALA-6201: Fix test_basic_filters on ASAN .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8646/1/tests/query_test/test_runtime_filters.py File tests/query_test/test_runtime_filters.py: http://gerrit.cloudera.org:8080/#/c/8646/1/tests/query_test/test_runtime_filters.py@27 PS1, Line 27: WAIT_TIME_MS = specific_build_type_timeout(6, slow_build_timeout=10) Just curious - did the longer timeout take effect on ASAN for you? I was seeing some cases locally where the ASAN build type detection logic flaked out on me and fell back to the DEBUG timing. -- To view, visit http://gerrit.cloudera.org:8080/8646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Gerrit-Change-Number: 8646 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Mon, 27 Nov 2017 22:46:23 + Gerrit-HasComments: Yes
[Impala-ASF-CR] IMPALA-6201: Fix test basic filters on ASAN
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8646 ) Change subject: IMPALA-6201: Fix test_basic_filters on ASAN .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Gerrit-Change-Number: 8646 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Sailesh Mukil Gerrit-Comment-Date: Mon, 27 Nov 2017 21:44:12 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6201: Fix test basic filters on ASAN
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8646 ) Change subject: IMPALA-6201: Fix test_basic_filters on ASAN .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/8646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Gerrit-Change-Number: 8646 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Sailesh Mukil Gerrit-Comment-Date: Mon, 27 Nov 2017 18:34:57 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6201: Fix test basic filters on ASAN
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8646 Change subject: IMPALA-6201: Fix test_basic_filters on ASAN .. IMPALA-6201: Fix test_basic_filters on ASAN TestRuntimeFilters.test_basic_filters is flaky on ASAN as sometimes the runtime filters aren't recieved within the specified RUNTIME_FILTER_WAIT_TIME_MS. This patch increases the timeout for ASAN builds. Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce --- M testdata/workloads/functional-query/queries/QueryTest/runtime_filters.test M tests/query_test/test_runtime_filters.py 2 files changed, 21 insertions(+), 20 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/46/8646/1 -- To view, visit http://gerrit.cloudera.org:8080/8646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I8c20cbb75a9b6da73137f220657aa75dea9dfdce Gerrit-Change-Number: 8646 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-Marshall