-
Notifications
You must be signed in to change notification settings - Fork 596
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
feat: support scaling in sql backend #14757
Conversation
22d8d98
to
40b99a0
Compare
src/meta/node/src/server.rs
Outdated
@@ -124,6 +124,7 @@ pub async fn rpc_serve( | |||
Some(backend) => { | |||
let mut options = sea_orm::ConnectOptions::new(backend.endpoint); | |||
options | |||
.sqlx_logging(false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here will be reverted before the PR is merged.
f7b0afb
to
d5c06d1
Compare
0cb5d21
to
ec1b8aa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx for the job, generally LGTM. We need to rely on some tests to help us discover some corner cases, as well as take advantage of the benefits of the SQL backend to make necessary optimizations. We can merge it now and do it in later PRs.
Merge queue setting changed
Signed-off-by: Shanicky Chen <[email protected]>
Signed-off-by: Shanicky Chen <[email protected]> Co-authored-by: Shanicky Chen <[email protected]>
I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
It supports low-level operations but doesn't yet support automatic scaling (lacking the field for table parallelism).
Checklist
./risedev check
(or alias,./risedev c
)