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

query exceeded per-node user memory after killed,resource not freed and service crashed #8341

Closed
ic4y opened this issue Jun 22, 2021 · 5 comments

Comments

@ic4y
Copy link

ic4y commented Jun 22, 2021

If a query’s resource occupation exceeds the resource of the node (Query exceeded per-node user memory limit of XXGB) , the query will be killed.

image

At this time, if you continue to submit the query that takes up a lot of resources (but it can be successfully executed normally). There will be insufficient resources, leading to continuous Major GC until the node hangs.

image

I suspect this is caused by the fact that the resource was not released after the query was killed.

The screenshot is a single machine test environment, and we also encountered such a problem in our production cluster.

@ic4y ic4y changed the title query exceeded per-node user memory after killed ,resource not freed query exceeded per-node user memory after killed,resource not freed and service crashed Jun 24, 2021
@ic4y
Copy link
Author

ic4y commented Jun 24, 2021

I need your help. Why is that

@sopel39
Copy link
Member

sopel39 commented Jun 28, 2021

Hi @ic4y

Which Trino version is this? There have been some improvements for releasing memory in operators recently.

@ic4y
Copy link
Author

ic4y commented Jun 29, 2021

Hi @sopel39 thanks for the reply.

I found this problem in both prestoSQl-322 and trino-353 versions.
Which version of the improvements you mentioned was done? I can do the corresponding test.

@sopel39
Copy link
Member

sopel39 commented Jun 29, 2021

Actually it will be in 359, see #8170 (comment)

@ic4y
Copy link
Author

ic4y commented Jul 28, 2021

@sopel39 Thanks . this solves the problem

@ic4y ic4y closed this as completed Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants