Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13770 )
Change subject: KUDU-2882: Increase the timeout value for TestSentryClientMetrics.Basic ...................................................................... Patch Set 1: Code-Review+1 (1 comment) In general, some tests are "flaky", meaning for various reasons, they may fail, just not all the time. We track such tests, and usually try to fix them if they go above a high flaky rate (e.g. >3% flaky rate). Flaky tests are tracked here: http://dist-test.cloudera.org:8080/ http://gerrit.cloudera.org:8080/#/c/13770/1/src/kudu/master/sentry_authz_provider-test.cc File src/kudu/master/sentry_authz_provider-test.cc: http://gerrit.cloudera.org:8080/#/c/13770/1/src/kudu/master/sentry_authz_provider-test.cc@1192 PS1, Line 1192: FLAGS_sentry_service_send_timeout_seconds = AllowSlowTests() ? 5 : 2; Perhaps do something like this, so the effect on runtime isn't as great? -- To view, visit http://gerrit.cloudera.org:8080/13770 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3f7f1dc0887b31c4e1a56071be2f0223aeb292e9 Gerrit-Change-Number: 13770 Gerrit-PatchSet: 1 Gerrit-Owner: helifu <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Wed, 17 Jul 2019 02:31:19 +0000 Gerrit-HasComments: Yes
