Skip to content

[core][rest] Support catalog-managed Format Table partitions - #8707

Merged
JingsongLi merged 4 commits into
apache:masterfrom
sundapeng:feat/format-table-managed-rest-contract
Jul 17, 2026
Merged

[core][rest] Support catalog-managed Format Table partitions#8707
JingsongLi merged 4 commits into
apache:masterfrom
sundapeng:feat/format-table-managed-rest-contract

Conversation

@sundapeng

@sundapeng sundapeng commented Jul 17, 2026

Copy link
Copy Markdown
Member

Purpose

Support catalog-managed partitions for Format Tables in REST Catalog. This adds create/drop APIs; managed partition listing does not fall back to filesystem discovery, and drops only unregister catalog metadata.

Tests

Added unit tests for REST serialization, managed and non-managed catalog behavior, and cache invalidation.

Define the managed partition wire format, fail-closed catalog APIs, and REST create/drop behavior. Keep server-side DROP metadata-only so data deletion remains a caller responsibility.
@sundapeng sundapeng changed the title [core][rest] Support catalog-managed Format Table partitions 【WIP】[core][rest] Support catalog-managed Format Table partitions Jul 17, 2026
private static final String FIELD_EXISTED = "existed";

@JsonProperty(FIELD_CREATED)
private final List<String> created;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not Map<String, String>?

@sundapeng sundapeng Jul 17, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

List<String> was used to return human-readable partition names such as dt=20260714/region=cn.
I think your suggestion List<Map<String, String>> is better and consistent with the request format:

{
  "created": [
    {
      "dt": "20260714",
      "region": "cn"
    },
    {
      "dt": "20260715",
      "region": "us"
    }
  ]
}

I'll update it, thank you for your comments.

@JingsongLi JingsongLi changed the title 【WIP】[core][rest] Support catalog-managed Format Table partitions [core][rest] Support catalog-managed Format Table partitions Jul 17, 2026
@JingsongLi

Copy link
Copy Markdown
Contributor

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants