Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add delete_groups API to AdminClient. (no-op in simulation) #235

Merged
merged 6 commits into from
Jan 9, 2025

Conversation

xxchan
Copy link
Contributor

@xxchan xxchan commented Jan 9, 2025

needed by risingwavelabs/risingwave#20065

Signed-off-by: xxchan xxchan22f@gmail.com

Signed-off-by: xxchan <xxchan22f@gmail.com>
@xxchan xxchan force-pushed the xxchan/gross-turkey branch from a61adf3 to d2b9fbb Compare January 9, 2025 06:24
Signed-off-by: xxchan <xxchan22f@gmail.com>
@xxchan
Copy link
Contributor Author

xxchan commented Jan 9, 2025

@BugenZhao @kwannoel

@xxchan
Copy link
Contributor Author

xxchan commented Jan 9, 2025

👀
image

@kwannoel
Copy link
Contributor

kwannoel commented Jan 9, 2025

Looks like an unrelated failure.

@kwannoel
Copy link
Contributor

kwannoel commented Jan 9, 2025

Hmm not sure what's the root cause, opened an issue: #236.

@kwannoel
Copy link
Contributor

kwannoel commented Jan 9, 2025

Maybe skip the test first in your PR @xxchan?

xxchan added 3 commits January 9, 2025 15:32
Signed-off-by: xxchan <xxchan22f@gmail.com>
Signed-off-by: xxchan <xxchan22f@gmail.com>
Signed-off-by: xxchan <xxchan22f@gmail.com>
@xxchan
Copy link
Contributor Author

xxchan commented Jan 9, 2025

Another broken test
image

@xxchan xxchan requested a review from kwannoel January 9, 2025 08:05
@kwannoel kwannoel merged commit aaa9f9a into madsim-rs:main Jan 9, 2025
12 checks passed
@xxchan
Copy link
Contributor Author

xxchan commented Jan 9, 2025

@kwannoel Can you publish the new version to crates.io? Thanks

@xxchan xxchan deleted the xxchan/gross-turkey branch January 9, 2025 10:24
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.

2 participants