Skip to content

Commit 00832bd

Browse files
committed
ignore check contain and add some comment
1 parent d54ae81 commit 00832bd

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

src/query/storages/system/src/tables_table.rs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -318,9 +318,7 @@ where TablesTable<WITH_HISTORY, WITHOUT_VIEW>: HistoryAware
318318
}
319319
} else if col_name == "name" {
320320
if let Scalar::String(t_name) = scalar {
321-
if !tables_names.contains(t_name) {
322-
tables_names.insert(t_name.clone());
323-
}
321+
tables_names.insert(t_name.clone());
324322
}
325323
}
326324
Ok(())
@@ -359,6 +357,7 @@ where TablesTable<WITH_HISTORY, WITHOUT_VIEW>: HistoryAware
359357
tables_names.extend(new_table_names);
360358
}
361359
Err(err) => {
360+
// swallow the errors related with mget tables
362361
warn!("Failed to get tables: {}, {}", ctl.name(), err);
363362
}
364363
}
@@ -438,9 +437,7 @@ where TablesTable<WITH_HISTORY, WITHOUT_VIEW>: HistoryAware
438437
{
439438
Ok(tables) => {
440439
for table in tables.into_iter().flatten() {
441-
if !tables_names.contains(&table) {
442-
tables_names.insert(table.clone());
443-
}
440+
tables_names.insert(table.clone());
444441
}
445442
}
446443
Err(err) => {

0 commit comments

Comments
 (0)