You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2019/07/19 22:02:00 UTC
[kudu] 01/02: KUDU-2882: Increase the timeout value for
TestSentryClientMetrics.Basic
This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit a61d88dbc6b9874f1628c0870c6db90104f513e8
Author: helifu <hz...@corp.netease.com>
AuthorDate: Mon Jul 1 18:34:23 2019 +0800
KUDU-2882: Increase the timeout value for TestSentryClientMetrics.Basic
It will be better to increase the timeout value to avoid the failures
in some test environments.
Change-Id: I3f7f1dc0887b31c4e1a56071be2f0223aeb292e9
Reviewed-on: http://gerrit.cloudera.org:8080/13770
Tested-by: Kudu Jenkins
Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
src/kudu/master/sentry_authz_provider-test.cc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/kudu/master/sentry_authz_provider-test.cc b/src/kudu/master/sentry_authz_provider-test.cc
index fd8230a..2c5efed 100644
--- a/src/kudu/master/sentry_authz_provider-test.cc
+++ b/src/kudu/master/sentry_authz_provider-test.cc
@@ -1393,8 +1393,8 @@ TEST_F(TestSentryClientMetrics, Basic) {
// Shorten the default timeout parameters: make timeout interval shorter.
NO_FATALS(sentry_authz_provider_->Stop());
FLAGS_sentry_service_rpc_addresses = sentry_->address().ToString();
- FLAGS_sentry_service_send_timeout_seconds = 2;
- FLAGS_sentry_service_recv_timeout_seconds = 2;
+ FLAGS_sentry_service_send_timeout_seconds = AllowSlowTests() ? 5 : 2;
+ FLAGS_sentry_service_recv_timeout_seconds = AllowSlowTests() ? 5 : 2;
sentry_authz_provider_.reset(new SentryAuthzProvider(metric_entity_));
ASSERT_OK(sentry_authz_provider_->Start());