-
Notifications
You must be signed in to change notification settings - Fork 753
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(query): support direct the spill data to a different bucket #17159
Conversation
f624450
to
06ddcac
Compare
src/query/service/src/pipelines/processors/transforms/transform_stream_sort_spill.rs
Show resolved
Hide resolved
We are modifying as much code as we are adding with It’s also difficult to document for users. For instance, there is no |
66a3274
to
0503d61
Compare
0503d61
to
7ec5a8d
Compare
databend/src/query/config/src/config.rs Lines 239 to 253 in 7ec5a8d
This is a better layout, but there's no time to refactor it. |
We need a test to cover that spill could work out with other storage params. Maybe add other test bucket like |
e265c73
to
999d12c
Compare
I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/
Summary
Tests
Type of change
This change is