fix bug for OLAP_SUCCESS with Status

apache · May 9, 2022 · fec4655 · fec4655
1 parent 806fcf7
commit fec4655
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/be/src/olap/storage_migration_v2.cpp b/be/src/olap/storage_migration_v2.cpp
@@ -355,8 +355,9 @@ Status StorageMigrationV2Handler::_convert_historical_rowsets(
         }
 
         if (!(res = _generate_rowset_writer(sm_params.base_tablet->tablet_path_desc(),
-                                           sm_params.new_tablet->tablet_path_desc(), rs_reader,
-                                           rowset_writer.get(), new_tablet)).ok()) {
+                                            sm_params.new_tablet->tablet_path_desc(), rs_reader,
+                                            rowset_writer.get(), new_tablet))
+                    .ok()) {
             LOG(WARNING) << "failed to add_rowset. version=" << rs_reader->version().first << "-"
                          << rs_reader->version().second;
             new_tablet->data_dir()->remove_pending_ids(ROWSET_ID_PREFIX +

diff --git a/be/src/olap/tablet_manager.cpp b/be/src/olap/tablet_manager.cpp
@@ -1068,8 +1068,9 @@ void TabletManager::try_delete_unused_tablet_path(DataDir* data_dir, TTabletId t
             if (!tablet_uid.empty() && !storage_name.empty()) {
                 segment_desc.storage_name = storage_name;
                 StorageParamPB storage_param;
-                if (!StorageBackendMgr::instance()->get_storage_param(
-                            storage_name, &storage_param).ok()) {
+                if (!StorageBackendMgr::instance()
+                            ->get_storage_param(storage_name, &storage_param)
+                            .ok()) {
                     LOG(WARNING) << "storage_name is invalid: " << storage_name;
                     return;
                 }