Skip to content

[VL] Celeborn shuffle reader OOM with many empty input stream#9221

Merged
marin-ma merged 1 commit into
apache:mainfrom
marin-ma:rss-reader-oom
Apr 4, 2025
Merged

[VL] Celeborn shuffle reader OOM with many empty input stream#9221
marin-ma merged 1 commit into
apache:mainfrom
marin-ma:rss-reader-oom

Conversation

@marin-ma

@marin-ma marin-ma commented Apr 3, 2025

Copy link
Copy Markdown
Contributor

Below code piece will create a buffered input stream on native side. The buffer is released on close() called when EOS is reached in readValue(). For empty input stream, the readValue function is never called, and the buffered memory can only be released at task end.

      ShuffleReaderJniWrapper
        .create(runtime)
        .readStream(shuffleReaderHandle, byteIn))
@github-actions github-actions Bot added the VELOX label Apr 3, 2025
@marin-ma marin-ma marked this pull request as draft April 3, 2025 16:39
@github-actions

github-actions Bot commented Apr 3, 2025

Copy link
Copy Markdown

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/apache/incubator-gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 participants