[Hotfix]Delete sync tables when drop external catalogs

ams/server/src/main/java/com/netease/arctic/server/persistence/mapper/TableMetaMapper.java

@@ -241,6 +241,9 @@ Integer deleteTableIdByName(
       @Param("databaseName") String databaseName,
       @Param("tableName") String tableName);
 
+  @Delete("DELETE FROM table_identifier WHERE catalog_name = #{catalogName}")
+  Integer deleteTableIdByCatalogName(@Param("catalogName") String catalogName);
+
   @Select(
       "SELECT table_id, catalog_name, db_name, table_name, format FROM table_identifier"
           + " WHERE catalog_name = #{catalogName} AND db_name = #{databaseName} AND table_name = #{tableName}")

ams/server/src/main/java/com/netease/arctic/server/table/DefaultTableService.java

@@ -140,12 +140,16 @@ private void initServerCatalog(CatalogMeta catalogMeta) {
   @Override
   public void dropCatalog(String catalogName) {
     checkStarted();
-    doAsExisted(
-        CatalogMetaMapper.class,
-        mapper -> mapper.deleteCatalog(catalogName),
+    doAsTransaction(
+        () ->
+            doAsExisted(
+                CatalogMetaMapper.class,
+                mapper -> mapper.deleteCatalog(catalogName),
+                () ->
+                    new IllegalMetadataException(
+                        "Catalog " + catalogName + " has more than one database or table")),
         () ->
-            new IllegalMetadataException(
-                "Catalog " + catalogName + " has more than one database or table"));
+            doAs(TableMetaMapper.class, mapper -> mapper.deleteTableIdByCatalogName(catalogName)));
     internalCatalogMap.remove(catalogName);
     externalCatalogMap.remove(catalogName);
   }