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)

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