[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. IMPALA-6242: Reduce flakiness in TimerCounterTest The error threshold in TimerCounterTest is 15ms, which is still not enough in some rare cases. This patch increases it to 30ms. Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Reviewed-on: http://gerrit.cloudera.org:8080/8670 Reviewed-by: Tim ArmstrongTested-by: Impala Public Jenkins --- M be/src/util/runtime-profile-test.cc 1 file changed, 16 insertions(+), 16 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 5 Gerrit-Owner: Tianyi Wang Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: Zoltan Borok-Nagy
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 4 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: Zoltan Borok-Nagy Gerrit-Comment-Date: Sat, 09 Dec 2017 03:58:22 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1603/ -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 4 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: Zoltan Borok-Nagy Gerrit-Comment-Date: Sat, 09 Dec 2017 00:22:13 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 4 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: Zoltan Borok-Nagy Gerrit-Comment-Date: Sat, 09 Dec 2017 00:22:18 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/8670/3/be/src/util/runtime-profile-test.cc File be/src/util/runtime-profile-test.cc: http://gerrit.cloudera.org:8080/#/c/8670/3/be/src/util/runtime-profile-test.cc@631 PS3, Line 631: DCHECK We should use EXPECT_TRUE. DCHECK crashes the process and is also removed in release builds. -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 3 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: Zoltan Borok-Nagy Gerrit-Comment-Date: Fri, 08 Dec 2017 16:33:35 + Gerrit-HasComments: Yes
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. Patch Set 3: Code-Review+1 Thanks for applying the changes. Seems OK to me. -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 3 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Zoltan Borok-Nagy Gerrit-Comment-Date: Fri, 08 Dec 2017 09:18:05 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Hello Zoltan Borok-Nagy, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8670 to look at the new patch set (#3). Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. IMPALA-6242: Reduce flakiness in TimerCounterTest The error threshold in TimerCounterTest is 15ms, which is still not enough in some rare cases. This patch increases it to 30ms. Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 --- M be/src/util/runtime-profile-test.cc 1 file changed, 14 insertions(+), 14 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/70/8670/3 -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 3 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Zoltan Borok-Nagy
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/8670/2/be/src/util/runtime-profile-test.cc File be/src/util/runtime-profile-test.cc: http://gerrit.cloudera.org:8080/#/c/8670/2/be/src/util/runtime-profile-test.cc@630 PS2, Line 630: if (csw_.TotalRunningTime() > 60) { > Might need to update this check as well. Done http://gerrit.cloudera.org:8080/#/c/8670/2/be/src/util/runtime-profile-test.cc@631 PS2, Line 631: DCHECK(false); > It wasn't introduced in this commit, but to my understanding EXPECT_FALSE(f Done -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 3 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Zoltan Borok-Nagy Gerrit-Comment-Date: Fri, 08 Dec 2017 01:29:14 + Gerrit-HasComments: Yes
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Tianyi Wang has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. IMPALA-6242: Reduce flakiness in TimerCounterTest The error threshold in TimerCounterTest is 15ms, which is still not enough in some rare cases. This patch increases it to 30ms. Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 --- M be/src/util/runtime-profile-test.cc 1 file changed, 12 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/70/8670/2 -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 2 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Zoltan Borok-Nagy
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8670 ) Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. Patch Set 1: Since we are measuring wall time here, I don't know any better. I think 30 ms for measurement error is still acceptable, but maybe you could increase the sleep times as well to keep the measurement error a small fraction of the measured runtime. -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Tianyi Wang Gerrit-Reviewer: Zoltan Borok-Nagy Gerrit-Comment-Date: Wed, 06 Dec 2017 16:37:32 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6242: Reduce flakiness in TimerCounterTest
Tianyi Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8670 Change subject: IMPALA-6242: Reduce flakiness in TimerCounterTest .. IMPALA-6242: Reduce flakiness in TimerCounterTest The error threshold in TimerCounterTest is 15ms, which is still not enough in some rare cases. This patch increases it to 30ms. Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 --- M be/src/util/runtime-profile-test.cc 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/70/8670/1 -- To view, visit http://gerrit.cloudera.org:8080/8670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ifc038908857060ccbabfe30c46e72fd93907f412 Gerrit-Change-Number: 8670 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi WangGerrit-Reviewer: Tianyi Wang