fix: fix compile error

4paradigm · Jul 21, 2023 · 90429e4 · 90429e4
1 parent 2677718
commit 90429e4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/storage/disk_table.cc b/src/storage/disk_table.cc
@@ -328,9 +328,9 @@ bool DiskTable::Delete(uint32_t idx, const std::string& pk,
     if (inner_index && inner_index->GetIndex().size() > 1) {
         auto ts_col = index_def->GetTsColumn();
         if (!ts_col) {
-            return {-1, "ts column not found"};
+            return false;
         }
-        combine_key1 = CombineKeyTs(pk, start_ts, ts_col->GetId());
+        combine_key1 = CombineKeyTs(pk, real_start_ts, ts_col->GetId());
         combine_key2 = CombineKeyTs(pk, real_end_ts, ts_col->GetId());
     } else {
         combine_key1 = CombineKeyTs(pk, real_start_ts);

diff --git a/src/storage/mem_table.cc b/src/storage/mem_table.cc
@@ -242,12 +242,10 @@ bool MemTable::Delete(const ::openmldb::api::LogEntry& entry) {
             if (!index_def || !index_def->IsReady()) {
                 continue;
             }
-            uint32_t real_idx = index_def->GetInnerPos();
             auto ts_col = index_def->GetTsColumn();
             if (!ts_col->IsAutoGenTs() && ts_col->GetName() != entry.ts_name()) {
                 continue;
             }
-            std::optional<uint32_t> ts_idx = ts_col ? std::optional<uint32_t>{ts_col->GetId()} : std::nullopt;
             uint32_t idx = index_def->GetId();
             std::unique_ptr<TraverseIterator> iter(NewTraverseIterator(idx));
             iter->SeekToFirst();