The Wayback Machine - https://web.archive.org/web/20211016054210/https://github.com/trinodb/trino/issues/9080
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

Jump to bottom

Cumulative user memory overflow, visible in the UI #9080

Open
leniartek opened this issue Sep 1, 2021 · 3 comments
Open

Cumulative user memory overflow, visible in the UI #9080

leniartek opened this issue Sep 1, 2021 · 3 comments
Labels
bug good first issue ui

Comments

@leniartek
Copy link

@leniartek leniartek commented Sep 1, 2021

Cumulative user memory in WebUI can display negative number for queries with very large memory consumption.

image
@findepi findepi changed the title [WebUI] Cumulative user memory display negative number Cumulative user memory overflow, visible in the UI Sep 2, 2021
@findepi
Copy link
Member

@findepi findepi commented Sep 2, 2021

@leniartek thanks for the report!
I changed the title on the assumption the overflow doesn't affect UI only, but may be visible for other consumers of the information.
cc @sopel39 @dain @aalbu
@anneloreegger
Copy link

@anneloreegger anneloreegger commented Oct 3, 2021

Could I work on this?
Would you want that the overflow is prevented, or should one then see on the UI a message like for example "too much memory consumption" instead of the negative number?
@sopel39
Copy link
Member

@sopel39 sopel39 commented Oct 4, 2021

Could I work on this?

Please do

Would you want that the overflow is prevented, or should one then see on the UI a message like for example "too much memory consumption" instead of the negative number?

Could we just use a number with bigger resolution?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug good first issue ui
4 participants
@findepi @sopel39 @leniartek @anneloreegger