You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by ad...@apache.org on 2020/01/21 22:41:11 UTC
[kudu] 03/04: rpc-test-base.h: squelch a warning
This is an automated email from the ASF dual-hosted git repository.
adar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit ceeabc5460bf4d791441bc1b2d37347475c1b399
Author: Adar Dembo <ad...@cloudera.com>
AuthorDate: Mon Jan 20 01:14:54 2020 -0800
rpc-test-base.h: squelch a warning
Change-Id: I4ae8ebb0cb49e32f08f5c3f0c0f5061b029e0638
Reviewed-on: http://gerrit.cloudera.org:8080/15079
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <as...@cloudera.com>
Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
src/kudu/rpc/rpc-test-base.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/kudu/rpc/rpc-test-base.h b/src/kudu/rpc/rpc-test-base.h
index 0b6f047..fcc7452 100644
--- a/src/kudu/rpc/rpc-test-base.h
+++ b/src/kudu/rpc/rpc-test-base.h
@@ -559,7 +559,9 @@ class RpcTestBase : public KuduTest {
// We shouldn't timeout significantly faster than our configured timeout, unless the
// rpc is cancelled.
- if (!will_be_cancelled) EXPECT_GE(elapsed_millis, expected_millis - 10);
+ if (!will_be_cancelled) {
+ EXPECT_GE(elapsed_millis, expected_millis - 10);
+ }
// And we also shouldn't take the full time that we asked for
EXPECT_LT(elapsed_millis * 1000, sleep_micros);
if (will_be_cancelled) {