From 26f7fb06e54c400640a61950a677e78c9da70578 Mon Sep 17 00:00:00 2001 From: Arpan Agrawal Date: Thu, 26 Jan 2023 20:25:26 +0530 Subject: [PATCH] Reply bug fixed: shouldn't check for table's existence --- src/yb/tablet/tablet_bootstrap.cc | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/src/yb/tablet/tablet_bootstrap.cc b/src/yb/tablet/tablet_bootstrap.cc index 74bd67c9af27..e7e948c4c5d2 100644 --- a/src/yb/tablet/tablet_bootstrap.cc +++ b/src/yb/tablet/tablet_bootstrap.cc @@ -1487,20 +1487,21 @@ class TabletBootstrap { consensus::LWReplicateMsg* replicate_msg, AlreadyAppliedToRegularDB already_applied_to_regular_db) { auto* request = replicate_msg->mutable_change_metadata_request(); - + LOG_WITH_FUNC(INFO) << "Inside PlayChangeMetadataRequest"; ChangeMetadataOperation operation(tablet_, log_.get(), request); // If table id isn't in metadata, ignore the replay as the table might've been dropped. - auto table_info = meta_->GetTableInfo(operation.table_id().ToBuffer()); - if (!table_info.ok()) { - LOG_WITH_PREFIX(WARNING) << "Table ID " << operation.table_id() - << " not found in metadata, skipping this ChangeMetadataRequest"; - return Status::OK(); - } + // auto table_info = meta_->GetTableInfo(operation.table_id().ToBuffer()); + // if (!table_info.ok()) { + // LOG_WITH_PREFIX(WARNING) << "Table ID " << operation.table_id() + // << " not found in metadata, skipping this ChangeMetadataRequest"; + // return Status::OK(); + // } RETURN_NOT_OK(operation.Prepare(IsLeaderSide::kTrue)); if (tablet_->metadata()->IsTableMetadataInRocksDB()) { + LOG_WITH_FUNC(INFO) << "Playing ChangeMetadataRequest"; operation.set_op_id(OpId::FromPB(replicate_msg->id())); HybridTime hybrid_time(replicate_msg->hybrid_time()); operation.set_hybrid_time(hybrid_time);