Skip to content

Conversation

@wardlican
Copy link
Contributor

Why are the changes needed?

In DefaultTable Service, it is compatible with master-slave mode. In master-slave mode, it is necessary to load tables from catalog, determine the addition and deletion of tables, complete the allocation of tables to bucketId, and the logic of ams nodes loading the tables they are responsible for based on bucketId.

Close #3923 .

Brief change log

  • Add a bucketTableSyncScheduler to DefaultTableService for table loading logic in master-slave mode.
  • The history of adding upgrades and switching to master-slave mode is that the allocation table is initialized with SQL for bucket allocation.

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
Copy link

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@amoro.apache.org list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Dec 15, 2025
@wardlican
Copy link
Contributor Author

Please perform CR on these implementations.

@github-actions github-actions bot removed the stale label Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Subtask]: Modify DefaultTableService to be compatible with master-slave mode

1 participant