fix(catalog): return 404 instead of 503 when drop races with concurrent delete (#4499)

* fix(catalog): return 404 instead of 503 when drop races with concurrent delete

When a table/view/namespace is concurrently deleted by another request,
dropEntityIfExists returns CATALOG_PATH_CANNOT_BE_RESOLVED. Previously
this fell through to the default case and threw ServiceFailureException
(HTTP 503). The entity is simply gone — this should be treated the same
as ENTITY_NOT_FOUND (return false / 404), not a service failure.

* Add DEBUG log when CATALOG_PATH_CANNOT_BE_RESOLVED on drop table

* Add DEBUG log for CATALOG_PATH_CANNOT_BE_RESOLVED in dropNamespace and dropView

Author: singhpk234

runtime/service/src/main/java/org/apache/polaris/service/catalog/iceberg/IcebergCatalog.java

@@ -434,6 +434,13 @@ public boolean dropTable(TableIdentifier tableIdentifier, boolean purge) {
         case BaseResult.ReturnStatus.ENTITY_NOT_FOUND:
           return false;
 
+        case BaseResult.ReturnStatus.CATALOG_PATH_CANNOT_BE_RESOLVED:
+          LOGGER.debug(
+              "Catalog path cannot be resolved for {}, treating as dropped; extraInfo={}",
+              tableIdentifier,
+              dropEntityResult.getExtraInformation());
+          return false;
+
         case BaseResult.ReturnStatus.ENTITY_UNDROPPABLE:
           throw new ForbiddenException(
               "Table %s cannot be dropped: %s",
@@ -683,6 +690,13 @@ public boolean dropNamespace(Namespace namespace) throws NamespaceNotEmptyExcept
         case BaseResult.ReturnStatus.ENTITY_NOT_FOUND:
           return false;
 
+        case BaseResult.ReturnStatus.CATALOG_PATH_CANNOT_BE_RESOLVED:
+          LOGGER.debug(
+              "Catalog path cannot be resolved for {}, treating as dropped; extraInfo={}",
+              namespace,
+              dropEntityResult.getExtraInformation());
+          return false;
+
         default:
           throw new ServiceFailureException(
               "Failed to drop namespace %s, status=%s, extraInfo=%s",
@@ -934,6 +948,13 @@ public boolean dropView(TableIdentifier identifier) {
         case BaseResult.ReturnStatus.ENTITY_NOT_FOUND:
           return false;
 
+        case BaseResult.ReturnStatus.CATALOG_PATH_CANNOT_BE_RESOLVED:
+          LOGGER.debug(
+              "Catalog path cannot be resolved for {}, treating as dropped; extraInfo={}",
+              identifier,
+              dropEntityResult.getExtraInformation());
+          return false;
+
         case BaseResult.ReturnStatus.ENTITY_UNDROPPABLE:
           throw new ForbiddenException(
               "View %s cannot be dropped: %s", identifier, dropEntityResult.getExtraInformation());

runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogTest.java

@@ -2083,6 +2083,70 @@ public void testDropViewWithUndroppableEntity() {
         .hasMessageContaining("cannot be dropped");
   }
 
+  @Test
+  public void testDropTableWithCatalogPathCannotBeResolved() {
+    catalog.createNamespace(NS);
+    catalog.buildTable(TABLE, SCHEMA).create();
+
+    PolarisMetaStoreManager spiedManager = spy(metaStoreManager);
+    doReturn(new DropEntityResult(BaseResult.ReturnStatus.CATALOG_PATH_CANNOT_BE_RESOLVED, null))
+        .when(spiedManager)
+        .dropEntityIfExists(any(), anyList(), any(), anyMap(), anyBoolean());
+
+    IcebergCatalog spiedCatalog = newIcebergCatalog(CATALOG_NAME, spiedManager, fileIOFactory);
+    spiedCatalog.initialize(
+        CATALOG_NAME,
+        ImmutableMap.of(
+            CatalogProperties.FILE_IO_IMPL, "org.apache.iceberg.inmemory.InMemoryFileIO"));
+
+    boolean result = spiedCatalog.dropTable(TABLE, false);
+    Assertions.assertThat(result).isFalse();
+  }
+
+  @Test
+  public void testDropViewWithCatalogPathCannotBeResolved() {
+    catalog.createNamespace(NS);
+    catalog
+        .buildView(TABLE)
+        .withSchema(SCHEMA)
+        .withDefaultNamespace(NS)
+        .withQuery("spark", "SELECT * FROM ns.tbl")
+        .create();
+
+    PolarisMetaStoreManager spiedManager = spy(metaStoreManager);
+    doReturn(new DropEntityResult(BaseResult.ReturnStatus.CATALOG_PATH_CANNOT_BE_RESOLVED, null))
+        .when(spiedManager)
+        .dropEntityIfExists(any(), anyList(), any(), anyMap(), anyBoolean());
+
+    IcebergCatalog spiedCatalog = newIcebergCatalog(CATALOG_NAME, spiedManager, fileIOFactory);
+    spiedCatalog.initialize(
+        CATALOG_NAME,
+        ImmutableMap.of(
+            CatalogProperties.FILE_IO_IMPL, "org.apache.iceberg.inmemory.InMemoryFileIO"));
+
+    boolean result = spiedCatalog.dropView(TABLE);
+    Assertions.assertThat(result).isFalse();
+  }
+
+  @Test
+  public void testDropNamespaceWithCatalogPathCannotBeResolved() {
+    catalog.createNamespace(NS);
+
+    PolarisMetaStoreManager spiedManager = spy(metaStoreManager);
+    doReturn(new DropEntityResult(BaseResult.ReturnStatus.CATALOG_PATH_CANNOT_BE_RESOLVED, null))
+        .when(spiedManager)
+        .dropEntityIfExists(any(), anyList(), any(), anyMap(), anyBoolean());
+
+    IcebergCatalog spiedCatalog = newIcebergCatalog(CATALOG_NAME, spiedManager, fileIOFactory);
+    spiedCatalog.initialize(
+        CATALOG_NAME,
+        ImmutableMap.of(
+            CatalogProperties.FILE_IO_IMPL, "org.apache.iceberg.inmemory.InMemoryFileIO"));
+
+    boolean result = spiedCatalog.dropNamespace(NS);
+    Assertions.assertThat(result).isFalse();
+  }
+
   private TableMetadata createSampleTableMetadata(String tableLocation) {
     Schema schema =
         new Schema(