kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject [kudu] 01/02: KUDU-2882: Increase the timeout value for TestSentryClientMetrics.Basic
Date Fri, 19 Jul 2019 22:02:00 GMT
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 <hzhelifu@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 <awong@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());
 


Mime
View raw message