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

Improve OOM message when a single reservation request fails to get more bytes. #11771

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

wiedld
Copy link
Contributor

@wiedld wiedld commented Aug 1, 2024

Which issue does this PR close?

Part of #11523

Rationale for this change

Improve the OOM error messages per here.

We have memory reservations, memory consumers, and a memory pool. The pool has many consumers, and each consumer can have many reservations. The code docs added in the previous PR make this delineation clear.

In this PR we changed the error message provided when we are comparing the pool vs reservation's request. We now explicitly define which bytes refer to the reservation vs the pool.

What changes are included in this PR?

Changes a single error message.

Are these changes tested?

Yes.

Are there any user-facing changes?

No API changes. Only less confusing error messages.

@wiedld wiedld marked this pull request as ready for review August 1, 2024 19:49
@alamb alamb changed the title Update OOM message when a single reservation request fails to get more bytes. Improve OOM message when a single reservation request fails to get more bytes. Aug 1, 2024
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @wiedld -- I think this message is much clearer than the previous one

@Dandandan Dandandan merged commit d010ce9 into apache:main Aug 2, 2024
29 checks passed
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.

4 participants