fix useless snapshot refresh in case no data in db (ydb-platform#4777)

zverevgeny · Jun 18, 2024 · a66a5fa · a66a5fa
1 parent 51e425d
commit a66a5fa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/ydb/services/metadata/ds_table/accessor_refresh.cpp b/ydb/services/metadata/ds_table/accessor_refresh.cpp
@@ -18,7 +18,7 @@ void TDSAccessorRefresher::OnNewEnrichedSnapshot(NFetcher::ISnapshot::TPtr snaps
 
 void TDSAccessorRefresher::OnNewParsedSnapshot(Ydb::Table::ExecuteQueryResult&& qResult, NFetcher::ISnapshot::TPtr snapshot) {
     *ProposedProto.mutable_result_sets() = std::move(*qResult.mutable_result_sets());
-    if (CurrentSelection.SerializeAsString() != ProposedProto.SerializeAsString()) {
+    if (!CurrentSnapshot || CurrentSelection.SerializeAsString() != ProposedProto.SerializeAsString()) {
         ALS_INFO(NKikimrServices::METADATA_PROVIDER) << "New refresher data: " << ProposedProto.DebugString();
         SnapshotConstructor->EnrichSnapshotData(snapshot, InternalController);
     } else {

diff --git a/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp b/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp
@@ -87,7 +87,7 @@ void TDSAccessorBase::Handle(TTableExistsActor::TEvController::TEvResult::TPtr&
         }
     }
     if (!hasExists) {
-        OnNewEnrichedSnapshot(SnapshotConstructor->CreateEmpty(RequestedActuality));
+        OnNewParsedSnapshot(Ydb::Table::ExecuteQueryResult(), SnapshotConstructor->CreateEmpty(RequestedActuality));
     } else {
         StartSnapshotsFetchingImpl();
     }