apache/gravitino

[Improvement] createTable may return success even when Gravitino metadata persistence fails

开放

#10,630 创建于 2026年4月1日

 (3 条评论) (0 个反应) (1 位负责人)Java (887 个派生)auto 404
good first issueimprovement

仓库指标

星标
 (3,058 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

What would you like to be improved?

In TableOperationDispatcher.createTable(...), Gravitino first creates the table in the underlying catalog, then writes the TableEntity to the entity store.

If store.put(tableEntity, true) fails, the exception is logged but swallowed, and the method still returns success.

This can leave the external table created, even though Gravitino metadata was not persisted. Clients will receive a successful create-table response even though Gravitino is in an inconsistent state.

Location is TableOperationDispatcher.java (line 640)

How should we improve?

Do not swallow the store.put(...) failure. Propagate the exception so create-table fails instead of returning false success OR if best-effort behavior is intended, return an explicit partial-success/inconsistent-state result and add a test for this path.

Add/modify an existing unit test to check for this.

贡献者指南