StarRocks / starrocks

StarRocks, a Linux Foundation project, is a next-generation sub-second MPP OLAP database for full analytics scenarios, including multi-dimensional analytics, real-time analytics, and ad-hoc queries.
https://starrocks.io
Apache License 2.0
8.27k stars 1.67k forks source link

[BugFix]remove check on num_rows and avail levels, batch skip to avoid big memory alloc (backport #47544) #47608

Open mergify[bot] opened 5 days ago

mergify[bot] commented 5 days ago

Why I'm doing:

to avoid big memory alloc, level_decoder return 1024*1024 level at most, so row_nums > avail_level is not suitable.

And to avoid big memory alloc, we'd better skip batch by batch(8096 in code).

What I'm doing:

Fixes #issue

What type of PR is this:

Does this PR entail a change in behavior?

If yes, please specify the type of change:

Checklist:

Bugfix cherry-pick branch check:

And to avoid big memory alloc, we'd better skip batch by batch(8096 in code).

What I'm doing:

Fixes #issue

What type of PR is this:

Does this PR entail a change in behavior?

If yes, please specify the type of change:

Checklist:

zombee0 commented 2 days ago

https://github.com/Mergifyio close

mergify[bot] commented 2 days ago

close

❌ Sorry but I didn't understand the command. Please consult the commands documentation 📚.