[Impala-ASF-CR] IMPALA-4659: fuzz test fixes
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4659: fuzz test fixes .. IMPALA-4659: fuzz test fixes * Apply a 512m mem_limit to all fuzz tests. This limits aggregate memory consumption to ~5GB per daemon(assuming 10 concurrent tests). * Refactor the exec option handling to use the exec_option dimension. This avoids executing the test multiple times redundantly * Remove the xfails to reduce noise, since there is no immediate plan to fix the product bugs. Instead just pass the tests. Testing: Ran in a loop for ~1h to flush out flakiness. Change-Id: Ie1942ceef252ec3e6171a0a54722b66a7d9abbd7 Reviewed-on: http://gerrit.cloudera.org:8080/5502 Reviewed-by: Tim Armstrong Tested-by: Impala Public Jenkins --- M tests/query_test/test_scanners_fuzz.py 1 file changed, 41 insertions(+), 25 deletions(-) Approvals: Impala Public Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/5502 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie1942ceef252ec3e6171a0a54722b66a7d9abbd7 Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Tim Armstrong
[Impala-ASF-CR] IMPALA-4659: fuzz test fixes
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4659: fuzz test fixes .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5502 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie1942ceef252ec3e6171a0a54722b66a7d9abbd7 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Tim Armstrong Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4659: fuzz test fixes
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4659: fuzz test fixes .. Patch Set 3: Code-Review+2 Carry +2 -- To view, visit http://gerrit.cloudera.org:8080/5502 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie1942ceef252ec3e6171a0a54722b66a7d9abbd7 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Tim Armstrong Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4659: fuzz test fixes
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4659: fuzz test fixes .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5502/2//COMMIT_MSG Commit Message: PS2, Line 7: IMPALA-4569 > wrong jira? Done -- To view, visit http://gerrit.cloudera.org:8080/5502 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie1942ceef252ec3e6171a0a54722b66a7d9abbd7 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Tim Armstrong Gerrit-HasComments: Yes
[Impala-ASF-CR] IMPALA-4659: fuzz test fixes
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5502 to look at the new patch set (#3). Change subject: IMPALA-4659: fuzz test fixes .. IMPALA-4659: fuzz test fixes * Apply a 512m mem_limit to all fuzz tests. This limits aggregate memory consumption to ~5GB per daemon(assuming 10 concurrent tests). * Refactor the exec option handling to use the exec_option dimension. This avoids executing the test multiple times redundantly * Remove the xfails to reduce noise, since there is no immediate plan to fix the product bugs. Instead just pass the tests. Testing: Ran in a loop for ~1h to flush out flakiness. Change-Id: Ie1942ceef252ec3e6171a0a54722b66a7d9abbd7 --- M tests/query_test/test_scanners_fuzz.py 1 file changed, 41 insertions(+), 25 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/02/5502/3 -- To view, visit http://gerrit.cloudera.org:8080/5502 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ie1942ceef252ec3e6171a0a54722b66a7d9abbd7 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Tim Armstrong