Skip to content

Conversation

@sollhui
Copy link
Contributor

@sollhui sollhui commented Aug 4, 2024

pick (#38255)

We meet OOM when using single stream multi table

image

It exist memory leak, and heap profile like:

image

The stream load context will not release in some exception conditions as plan failed for high concurrency causing timeout when obtaining read lock. It is introduced by #35458

The solution effect is shown in the following figure, which can run stably with a small amount of memory

image

We meet OOM when using single stream multi table

![image](https://github.com/user-attachments/assets/748e9914-d591-4f41-8b28-412d3cecc841)

It exist memory leak, and heap profile like:

![image](https://github.com/user-attachments/assets/af30c593-88ea-44f6-bba1-82436b13f99f)

The stream load context will not release in some exception conditions as
plan failed for high concurrency causing timeout when obtaining read
lock. It is introduced by #35458

The solution effect is shown in the following figure, which can run
stably with a small amount of memory

![image](https://github.com/user-attachments/assets/4483e0a5-6c0c-4cdc-b8ed-3408da6a86b2)
@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.

@sollhui
Copy link
Contributor Author

sollhui commented Aug 4, 2024

run buildall

@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2024

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

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 36.47% (9276/25433)
Line Coverage: 27.98% (75847/271049)
Region Coverage: 26.84% (39032/145430)
Branch Coverage: 23.55% (19807/84096)
Coverage Report: http://coverage.selectdb-in.cc/coverage/a779eede37777202c706953112a9ec82dc501c08_a779eede37777202c706953112a9ec82dc501c08/report/index.html

@dataroaring dataroaring merged commit 6035eda into apache:branch-2.1 Aug 4, 2024
@yiguolei yiguolei mentioned this pull request Sep 5, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants