small fix

XiaoMi · Feb 18, 2020 · 3d09954 · 3d09954
1 parent aa4740b
commit 3d09954
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/include/dsn/dist/replication/replication_app_base.h b/include/dsn/dist/replication/replication_app_base.h
@@ -228,7 +228,7 @@ class replication_app_base : public replica_base
     // query app envs.
     virtual void query_app_envs(/*out*/ std::map<std::string, std::string> &envs) = 0;
 
-    virtual void set_partition_version(uint32_t partition_version) {}
+    virtual void set_partition_version(int32_t partition_version) {}
 
 public:
     //

diff --git a/src/dist/replication/lib/replica.h b/src/dist/replication/lib/replica.h
@@ -496,7 +496,7 @@ class replica : public serverlet<replica>, public ref_counter, public replica_ba
     ballot _child_init_ballot{0};
     // in normal case _partition_version = partition_count-1
     // when parent register child, _partition_version = -1, will reject client read write request
-    std::atomic<int> _partition_version;
+    std::atomic<int32_t> _partition_version;
 
     // perf counters
     perf_counter_wrapper _counter_private_log_size;

diff --git a/src/dist/replication/test/replica_test/unit_test/mock_utils.h b/src/dist/replication/test/replica_test/unit_test/mock_utils.h
@@ -69,7 +69,7 @@ class mock_replication_app_base : public replication_app_base
     void query_app_envs(std::map<std::string, std::string> &out) override { out = _envs; }
     decree last_durable_decree() const override { return 0; }
 
-    void set_partition_version(uint32_t partition_version) override {}
+    void set_partition_version(int32_t partition_version) override {}
 
 private:
     std::map<std::string, std::string> _envs;