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

[Plasma] Incorrect log message when evicting objects #20868

Closed
asfimport opened this issue Jan 19, 2019 · 2 comments
Closed

[Plasma] Incorrect log message when evicting objects #20868

asfimport opened this issue Jan 19, 2019 · 2 comments

Comments

@asfimport
Copy link
Collaborator

When Plasma evicts objects on running out of memory, it prints log messages of the form:

There is not enough space to create this object, so evicting x objects to free up y bytes. The number of bytes in use (before this eviction) is z.

However, the reported number of bytes in use (before this eviction) actually reports the number of bytes after the eviction. A straightforward fix is to simply replace z with (y+z).

Reporter: Anurag Khandelwal / @anuragkh
Assignee: Anurag Khandelwal / @anuragkh

PRs and other links:

Note: This issue was originally created as ARROW-4295. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

Anurag Khandelwal / @anuragkh:
cc @pcmoritz

@asfimport
Copy link
Collaborator Author

Philipp Moritz / @pcmoritz:
Issue resolved by pull request 3433
#3433

@asfimport asfimport added this to the 0.13.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant