refactor: rm format_version in put

src/client/tablet_client.cc

@@ -241,17 +241,11 @@ bool TabletClient::UpdateTableMetaForAddField(uint32_t tid, const std::vector<op
 
 bool TabletClient::Put(uint32_t tid, uint32_t pid, uint64_t time, const std::string& value,
                        const std::vector<std::pair<std::string, uint32_t>>& dimensions) {
-    return Put(tid, pid, time, value, dimensions, 0);
-}
-
-bool TabletClient::Put(uint32_t tid, uint32_t pid, uint64_t time, const std::string& value,
-                       const std::vector<std::pair<std::string, uint32_t>>& dimensions, uint32_t format_version) {
     ::openmldb::api::PutRequest request;
     request.set_time(time);
     request.set_value(value);
     request.set_tid(tid);
     request.set_pid(pid);
-    request.set_format_version(format_version);
     for (size_t i = 0; i < dimensions.size(); i++) {
         ::openmldb::api::Dimension* d = request.add_dimensions();
         d->set_key(dimensions[i].first);
@@ -267,17 +261,15 @@ bool TabletClient::Put(uint32_t tid, uint32_t pid, uint64_t time, const std::str
     return false;
 }
 
-bool TabletClient::Put(uint32_t tid, uint32_t pid, const char* pk, uint64_t time, const char* value, uint32_t size,
-                       uint32_t format_version) {
+bool TabletClient::Put(uint32_t tid, uint32_t pid, const std::string& pk, uint64_t time, const std::string& value) {
     ::openmldb::api::PutRequest request;
     auto dim = request.add_dimensions();
     dim->set_key(pk);
     dim->set_idx(0);
     request.set_time(time);
-    request.set_value(value, size);
+    request.set_value(value);
     request.set_tid(tid);
     request.set_pid(pid);
-    request.set_format_version(format_version);
     ::openmldb::api::PutResponse response;
 
     bool ok =
@@ -289,11 +281,6 @@ bool TabletClient::Put(uint32_t tid, uint32_t pid, const char* pk, uint64_t time
     return false;
 }
 
-bool TabletClient::Put(uint32_t tid, uint32_t pid, const std::string& pk, uint64_t time, const std::string& value,
-                       uint32_t format_version) {
-    return Put(tid, pid, pk.c_str(), time, value.c_str(), value.size(), format_version);
-}
-
 bool TabletClient::MakeSnapshot(uint32_t tid, uint32_t pid, uint64_t offset, std::shared_ptr<TaskInfo> task_info) {
     ::openmldb::api::GeneralRequest request;
     request.set_tid(tid);

src/client/tablet_client.h

@@ -78,20 +78,11 @@ class TabletClient : public Client {
                               std::shared_ptr<::openmldb::sdk::SQLRequestRowBatch>, brpc::Controller* cntl,
                               ::openmldb::api::SQLBatchRequestQueryResponse* response, const bool is_debug = false);
 
-    bool Put(uint32_t tid, uint32_t pid, const std::string& pk, uint64_t time, const std::string& value,
-             uint32_t format_version = 0);
-
-    bool Put(uint32_t tid, uint32_t pid, const char* pk, uint64_t time, const char* value, uint32_t size,
-             uint32_t format_version = 0);
+    bool Put(uint32_t tid, uint32_t pid, const std::string& pk, uint64_t time, const std::string& value);
 
     bool Put(uint32_t tid, uint32_t pid, uint64_t time, const std::string& value,
              const std::vector<std::pair<std::string, uint32_t>>& dimensions);
 
-    bool Put(uint32_t tid, uint32_t pid, uint64_t time, const std::string& value,
-             const std::vector<std::pair<std::string, uint32_t>>& dimensions, uint32_t format_version);
-
-
-
     bool Get(uint32_t tid, uint32_t pid, const std::string& pk, uint64_t time, std::string& value,  // NOLINT
              uint64_t& ts,                                                                          // NOLINT
              std::string& msg);                        ;                                             // NOLINT

src/cmd/openmldb.cc

@@ -315,7 +315,7 @@ int PutData(uint32_t tid, const std::map<uint32_t, std::vector<std::pair<std::st
             }
         }
 
-        if (!clients[endpoint]->Put(tid, pid, ts, value, iter->second, format_version)) {
+        if (!clients[endpoint]->Put(tid, pid, ts, value, iter->second)) {
             printf("put failed. tid %u pid %u endpoint %s ts %lu \n", tid, pid, endpoint.c_str(), ts);
             return -1;
         }

src/sdk/mini_cluster_batch_bm.cc

@@ -97,7 +97,7 @@ static void BM_SimpleQueryFunction(benchmark::State& state) {  // NOLINT
     uint32_t tid = sdk.GetTableId(db, name);
     {
         for (int32_t i = 0; i < 1000; i++) {
-            ok = tablet[0]->GetClient()->Put(tid, 0, pk, ts + i, value, 1);
+            ok = tablet[0]->GetClient()->Put(tid, 0, pk, ts + i, value);
         }
     }
     std::string sql = "select col1, col2 + 1, col3, col4, col5 from " + name + " ;";

src/sdk/sql_cluster_router.cc

@@ -1192,7 +1192,7 @@ bool SQLClusterRouter::PutRow(uint32_t tid, const std::shared_ptr<SQLInsertRow>&
                 if (client) {
                     DLOG(INFO) << "put data to endpoint " << client->GetEndpoint() << " with dimensions size "
                                << kv.second.size();
-                    bool ret = client->Put(tid, pid, cur_ts, row->GetRow(), kv.second, 1);
+                    bool ret = client->Put(tid, pid, cur_ts, row->GetRow(), kv.second);
                     if (!ret) {
                         status->msg = "fail to make a put request to table. tid " + std::to_string(tid);
                         LOG(WARNING) << status->msg;

src/tablet/tablet_impl.cc

@@ -458,7 +458,7 @@ int32_t TabletImpl::GetIndex(const ::openmldb::api::GetRequest* request, const :
     }
     bool enable_project = false;
     openmldb::codec::RowProject row_project(vers_schema, request->projection());
-    if (request->projection().size() > 0 && meta.format_version() == 1) {
+    if (request->projection().size() > 0) {
         if (meta.compress_type() == ::openmldb::type::kSnappy) {
             return -1;
         }