Skip to content

[gateway] Add catalog and DDL backend operations with REST APIs - #4107

Merged
fresh-borzoni merged 7 commits into
apache:mainfrom
beryllw:gateway/catalog-ddl
Aug 27, 2026
Merged

fresh-borzoni merged 7 commits into
apache:mainfrom
beryllw:gateway/catalog-ddl

Conversation

@beryllw

@beryllw beryllw commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #4082

Brief change log

Tests

API and Format

Documentation

@beryllw
beryllw force-pushed the gateway/catalog-ddl branch from c1cfe30 to ffbb879 Compare August 26, 2026 06:26
@beryllw

beryllw commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

@fresh-borzoni fresh-borzoni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@beryllw Thank you for the PR, LGTM overall, a couple of comments, PTAL

Comment thread fluss-gateway/src/backend/errors.rs Resolved
Comment thread fluss-gateway/src/backend/errors.rs Resolved
Comment thread fluss-gateway/src/protocol/rest/ddl.rs Resolved
Comment thread fluss-gateway/src/protocol/rest/ddl.rs Resolved
Comment thread fluss-gateway/src/protocol/rest/ddl.rs Resolved

@naivedogger naivedogger left a comment

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.

Thanks @beryllw for this. Overall LGTM, just some minor comments, PTAL.

Comment thread fluss-gateway/src/backend/client.rs Resolved
Comment thread fluss-gateway/src/protocol/rest/ddl.rs Resolved
@beryllw
beryllw requested a review from fresh-borzoni August 26, 2026 15:39

@fresh-borzoni fresh-borzoni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@beryllw Thank you, LGTM overall, couple of minors, PTAL

Comment thread fluss-gateway/src/protocol/rest/ddl.rs Resolved
Comment thread fluss-gateway/src/backend/errors.rs Resolved
@beryllw
beryllw requested a review from fresh-borzoni August 27, 2026 05:48

@fresh-borzoni fresh-borzoni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@beryllw Thank you, LGTM πŸ‘

@fresh-borzoni
fresh-borzoni merged commit 4172162 into apache:main Aug 27, 2026
15 checks passed
@leonardBang
leonardBang self-requested a review August 27, 2026 06:37
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.

[FIP-49][gateway] Add catalog and DDL backend operations with REST APIs

3 participants