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

[FEA] Limit host memory usage on python UDF execs. #8903

Open
revans2 opened this issue Aug 1, 2023 · 0 comments
Open

[FEA] Limit host memory usage on python UDF execs. #8903

revans2 opened this issue Aug 1, 2023 · 0 comments
Labels
reliability Features to improve reliability or bugs that severly impact the reliability of the plugin task Work required that improves the product but is not user facing

Comments

@revans2
Copy link
Collaborator

revans2 commented Aug 1, 2023

Is your feature request related to a problem? Please describe.

#9862 should add a hard limit to most off heap host memory used.

This might get to be a little complicated. Internally this goes from CUDF GPU to Arrow CPU and from Arrow CPU across the wire to python. We also reverse it coming back. There is a small 10 MiB buffer allocated by java for this, but that is more of a bounce buffer. The CPU data allocated for arrow is where most of the memory would come from. I honestly don't know if we can insert some way to have arrow allocate the data from what we want and to reserve it/spill. Probably not. If we cannot figure this out, then we probably need to do something with the overhead for the tasks to try and account for this, or at least document it in some way.

If we can change how the allocation works, then we should add in retry blocks around those allocations. If we cannot, which is likely the case, then we need to document how this all works.

@revans2 revans2 added feature request New feature or request ? - Needs Triage Need team to review and classify task Work required that improves the product but is not user facing reliability Features to improve reliability or bugs that severly impact the reliability of the plugin and removed feature request New feature or request labels Aug 1, 2023
@mattahrens mattahrens removed the ? - Needs Triage Need team to review and classify label Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reliability Features to improve reliability or bugs that severly impact the reliability of the plugin task Work required that improves the product but is not user facing
Projects
None yet
Development

No branches or pull requests

2 participants