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/08/10 23:55:17 UTC

[kudu] 01/02: [kserver] fix const-correctness for getter methods

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 2187f9c1c8d8b98ffbeb464f16d2f52f8c76d3ba
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Sun Aug 9 21:11:27 2020 -0700

    [kserver] fix const-correctness for getter methods
    
    Changed signature of the getter methods in kserver to be const-correct
    as per [1].
    
    This patch does not contain any functional changes.
    
    [1] https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/const.md
    
    Change-Id: Id845e36ab99b3b9bd5c4ccb5eb6263a96acd5a0a
    Reviewed-on: http://gerrit.cloudera.org:8080/16316
    Tested-by: Alexey Serbin <as...@cloudera.com>
    Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
 src/kudu/kserver/kserver.h | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/kudu/kserver/kserver.h b/src/kudu/kserver/kserver.h
index f8b127f..37a7ffc 100644
--- a/src/kudu/kserver/kserver.h
+++ b/src/kudu/kserver/kserver.h
@@ -59,13 +59,12 @@ class KuduServer : public server::ServerBase {
   // Shuts down a KuduServer instance.
   void Shutdown() override;
 
-  ThreadPool* tablet_prepare_pool() const { return tablet_prepare_pool_.get(); }
-  ThreadPool* tablet_apply_pool() const { return tablet_apply_pool_.get(); }
-  ThreadPool* raft_pool() const { return raft_pool_.get(); }
-  scoped_refptr<AtomicGauge<int32_t>> num_raft_leaders() const { return num_raft_leaders_; }
+  ThreadPool* tablet_prepare_pool() { return tablet_prepare_pool_.get(); }
+  ThreadPool* tablet_apply_pool() { return tablet_apply_pool_.get(); }
+  ThreadPool* raft_pool() { return raft_pool_.get(); }
+  scoped_refptr<AtomicGauge<int32_t>> num_raft_leaders() { return num_raft_leaders_; }
 
  private:
-
   // Thread pool for preparing ops, shared between all tablets.
   std::unique_ptr<ThreadPool> tablet_prepare_pool_;