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 2020/09/16 19:19:24 UTC
[kudu] branch master updated: [mini_ranger] bump start timeout to
90 sec
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
The following commit(s) were added to refs/heads/master by this push:
new f63ca3b [mini_ranger] bump start timeout to 90 sec
f63ca3b is described below
commit f63ca3b6e7a61fbb0f56664d772e4e9237475671
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Tue Sep 15 22:15:13 2020 -0700
[mini_ranger] bump start timeout to 90 sec
In environments with limited amount of memory (Lenovo T420s laptop
with 8 GByte SDRAM with few XServer apps running), MiniRanger takes very
time long to start if multiple tests are running in parallel (ctest -j4):
I0913 09:13:32.260512 1891621 mini_ranger.cc:234] Ranger bound to 45529
I0913 09:13:32.260557 1891621 mini_ranger.cc:235] Ranger admin URL: http://127.0.0.1:48609
I0913 09:13:32.260591 1891621 mini_ranger.cc:163] Time spent starting Ranger: real 62.045s user 0.031s sys 0.134s
In most cases there, Ranger could not make it in happen in 60 seconds
while running mini_ranger-test. Increasing the timeout interval to 90
seconds helped.
Change-Id: I5e1ad3318a61376c1653ac14eadbe3d112c495f5
Reviewed-on: http://gerrit.cloudera.org:8080/16457
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke <gr...@apache.org>
---
src/kudu/ranger/mini_ranger.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/kudu/ranger/mini_ranger.cc b/src/kudu/ranger/mini_ranger.cc
index 35d90ce..b060ff0 100644
--- a/src/kudu/ranger/mini_ranger.cc
+++ b/src/kudu/ranger/mini_ranger.cc
@@ -46,7 +46,7 @@ using std::string;
using std::vector;
using strings::Substitute;
-static constexpr int kRangerStartTimeoutMs = 60000;
+static constexpr int kRangerStartTimeoutMs = 90000;
namespace kudu {
namespace ranger {