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

[opt](split) get file splits in batch mode #35107

Merged
merged 1 commit into from
May 21, 2024

Conversation

AshinGau
Copy link
Member

backport: #34032

Proposed changes

Issue Number: close #xxx

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@AshinGau
Copy link
Member Author

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@AshinGau AshinGau force-pushed the split_source_br2 branch 2 times, most recently from 127ab16 to 85281ac Compare May 21, 2024 03:57
@AshinGau
Copy link
Member Author

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

When scanning a table with many files, It will take a lot of time to transfer splits to backends.(20s of the following 1209172 splits).  Therefore, using batch mode to fetch the file splits, BE can do scanning while fetch the file splits.
@AshinGau
Copy link
Member Author

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@morningman morningman merged commit 98f8eb5 into apache:branch-2.1 May 21, 2024
15 of 17 checks passed
@morningman morningman mentioned this pull request Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants