Skip to content

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

Merged
morningman merged 1 commit into
apache:branch-2.1from
AshinGau:split_source_br2
May 21, 2024
Merged

[opt](split) get file splits in batch mode#35107
morningman merged 1 commit into
apache:branch-2.1from
AshinGau:split_source_br2

Conversation

@AshinGau

Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
MemberAuthor

run buildall

@github-actions

Copy link
Copy Markdown
Contributor

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

@AshinGau
AshinGauforce-pushed the split_source_br2 branch 2 times, most recently from 127ab16 to 85281acCompareMay 21, 2024 03:57
@AshinGau

Copy link
Copy Markdown
MemberAuthor

run buildall

@github-actions

Copy link
Copy Markdown
Contributor

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

1 similar comment
@github-actions

Copy link
Copy Markdown
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
Copy Markdown
MemberAuthor

run buildall

@github-actions

Copy link
Copy Markdown
Contributor

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

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@AshinGau@doris-robot@morningman