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) {