address comment

Author: jeewonhh

src/catalog/catalog.cpp

@@ -952,16 +952,14 @@ optional_ptr<SchemaCatalogEntry> Catalog::GetSchema(CatalogEntryRetriever &retri
                                                     const string &schema_name, OnEntryNotFound if_not_found,
                                                     QueryErrorContext error_context) {
 	auto entries = GetCatalogEntries(retriever, catalog_name, schema_name);
-	auto &context = retriever.GetContext();
 	for (idx_t i = 0; i < entries.size(); i++) {
-		auto entry_name = IsInvalidCatalog(entries[i].catalog) ? GetDefaultCatalog(retriever) : entries[i].catalog;
-		// check whether the catalog is attached before trying to get it
-		if (!context.db->GetDatabaseManager().GetDatabase(context, entry_name)) {
+		auto catalog = Catalog::GetCatalogEntry(retriever, entries[i].catalog);
+		if (!catalog) {
+			// skip if it is not an attached database
 			continue;
 		}
 		auto on_not_found = i + 1 == entries.size() ? if_not_found : OnEntryNotFound::RETURN_NULL;
-		auto &catalog = Catalog::GetCatalog(retriever, entries[i].catalog);
-		auto result = catalog.GetSchema(context, schema_name, on_not_found, error_context);
+		auto result = catalog->GetSchema(retriever.GetContext(), schema_name, on_not_found, error_context);
 		if (result) {
 			return result;
 		}