Skip to content

Commit

Permalink
[BACKPORT 2.20][#22037] docdb: Remove log spam generated by deleted c…
Browse files Browse the repository at this point in the history
…olocated tables

Summary:
When a colocated table is deleted, the metadata entity holding its metadata has its tablet maps cleared. However the metadata entry is never removed from the tables_ map in the catalog manager. There is a background task that loops through this tables_ map and calls a method which prints a log line when a colocated table has no backing tablet. YBM has run into issues with log spam due to this behaviour.

This diff simply removes the log line. I filed #23721 to track removing tables from the tables_ map on deletion.
Jira: DB-10954

Original commit: bb74fb6 / D37646

Test Plan: n/a

Reviewers: mhaddad

Reviewed By: mhaddad

Subscribers: ybase, slingam

Differential Revision: https://phorge.dev.yugabyte.com/D37687
  • Loading branch information
druzac committed Sep 6, 2024
1 parent 5f40002 commit 02af54f
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/yb/master/catalog_entity_info.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1027,7 +1027,6 @@ TabletInfoPtr TableInfo::GetColocatedUserTablet() const {
if (!tablets_.empty()) {
return tablets_.begin()->second;
}
LOG(INFO) << "Colocated Tablet not found for table " << name();
return nullptr;
}

Expand Down

0 comments on commit 02af54f

Please sign in to comment.