clusterpedia icon indicating copy to clipboard operation
clusterpedia copied to clipboard

Add message queue to implement multi-cluster watch interface When using the default storage layer

Open wuyingjun-lucky opened this issue 1 year ago • 8 comments

What type of PR is this? /kind design

What this PR does / why we need it:

Which issue(s) this PR fixes: Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Add message queue to implement multi-cluster watch interface

wuyingjun-lucky avatar Nov 15 '23 08:11 wuyingjun-lucky

Hi @wuyingjun-lucky, Thanks for your pull request! If the PR is ready, use the /auto-cc command to assign Reviewer to Review. We will review it shortly.

Details

Instructions for interacting with me using comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the gh-ci-bot repository.

clusterpedia-bot avatar Nov 15 '23 08:11 clusterpedia-bot

Fix https://github.com/clusterpedia-io/clusterpedia/issues/595

wuyingjun-lucky avatar Nov 17 '23 07:11 wuyingjun-lucky

/cc @Iceber

wuyingjun-lucky avatar Nov 21 '23 05:11 wuyingjun-lucky

Ready to be first round

wuyingjun-lucky avatar Nov 21 '23 05:11 wuyingjun-lucky

I noticed that this pr includes splitting the database tables

@calvin0327 is also moving forward with this recently, and we can discuss splitting tables further in the online meeting. https://github.com/clusterpedia-io/clusterpedia/issues/601

Iceber avatar Nov 29 '23 01:11 Iceber

I noticed that this pr includes splitting the database tables

@calvin0327 is also moving forward with this recently, and we can discuss splitting tables further in the online meeting. #601

Do we need to remove the splitting tables or wait for the next meeting ?

wuyingjun-lucky avatar Nov 30 '23 09:11 wuyingjun-lucky

I think the way the table is automatically split in this pr also makes a lot of sense, we can keep it in this pr for now and then discuss it as an example

Iceber avatar Dec 01 '23 06:12 Iceber

/cc @Iceber

wuyingjun-lucky avatar Dec 28 '23 02:12 wuyingjun-lucky