[Bug report] drop kafka catalog maybe failed after altering

[mchades]

Version

main branch

Describe what's wrong

drop kafka catalog maybe failed after altering

Error message and/or stacktrace

2024-05-30 14:25:24.197 [Gravitino-webserver-37] ERROR com.datastrato.gravitino.server.web.rest.ExceptionHandlers - Failed to operate object [test_catalog_836a3008] operation [DROP] under [catalogKafkaIT_metalake_94b63407], reason [Entity catalogKafkaIT_metalake_94b63407.test_catalog_836a3008 has sub-entities, you should remove sub-entities first]
com.datastrato.gravitino.exceptions.NonEmptyEntityException: Entity catalogKafkaIT_metalake_94b63407.test_catalog_836a3008 has sub-entities, you should remove sub-entities first
	at com.datastrato.gravitino.storage.relational.service.CatalogMetaService.deleteCatalog(CatalogMetaService.java:203) ~[gravitino-core-0.5.1-SNAPSHOT.jar:?]
	at com.datastrato.gravitino.storage.relational.JDBCBackend.delete(JDBCBackend.java:179) ~[gravitino-core-0.5.1-SNAPSHOT.jar:?]

How to reproduce

• create a kafka catalog
• alter the catalog
• drop the catalog

Additional context

CatalogKafkaIT accidentally triggered

CatalogKafkaIT > testCatalog() FAILED
    org.opentest4j.AssertionFailedError: expected: <true> but was: <false>
        at app//org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:55)
        at app//org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:40)
        at app//org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:35)
        at app//org.junit.jupiter.api.Assertions.assertTrue(Assertions.java:179)
        at app//com.datastrato.gravitino.catalog.kafka.integration.test.CatalogKafkaIT.testCatalog(CatalogKafkaIT.java:142)

It's because catalogOps was not initialized originally, and the asynchronous thread that eliminated the catalog cache made another initialization to close catalogOps, resulting in the default schema being created again.

The deletion logic of the Kafka catalog is as follows：

• The initialization of "close" is performed after line 514.
• An exception is thrown at line 518.
  https://github.com/datastrato/gravitino/blob/a09420dd90a413a4f4944fc628286d20f9950d1c/core/src/main/java/com/datastrato/gravitino/catalog/CatalogManager.java#L499-L525