Skip to content

[AMORO-2810] Support rest service for follower AMS#3737

Merged
zhoujinsong merged 6 commits intoapache:masterfrom
zhoujinsong:ha-rest
Aug 20, 2025
Merged

[AMORO-2810] Support rest service for follower AMS#3737
zhoujinsong merged 6 commits intoapache:masterfrom
zhoujinsong:ha-rest

Conversation

@zhoujinsong
Copy link
Contributor

Why are the changes needed?

This PR started the rest service for all AMS, including followers.
This is a subtask of #2810, and replace #3567.

Brief change log

  • Start rest service for follower AMS.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@github-actions github-actions bot added the module:ams-server Ams server module label Aug 19, 2025
@zhoujinsong
Copy link
Contributor Author

Can you help review this PR? @czy006 @klion26

@zhoujinsong
Copy link
Contributor Author

I tested the follower rest service in my local environment, including:

  • creating a group
  • creating a catalog
  • creating a table and querying the table within the terminal

Copy link
Contributor

@czy006 czy006 left a comment

Choose a reason for hiding this comment

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

We have already implemented this function in production,Thanks @zhoujinsong

@zhoujinsong zhoujinsong merged commit 75f92c8 into apache:master Aug 20, 2025
6 checks passed
@zhoujinsong zhoujinsong deleted the ha-rest branch August 20, 2025 02:40
@zhoujinsong
Copy link
Contributor Author

Thanks for the review! @czy006

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

Labels

module:ams-server Ams server module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants