Skip to content

[AURON #2116] Make libInputStream auto-close when creating in Flink/Spark AuronAdaptor.#2117

Merged
Tartarus0zm merged 1 commit intoapache:masterfrom
RocMarshal:AURON-2116
Mar 26, 2026
Merged

[AURON #2116] Make libInputStream auto-close when creating in Flink/Spark AuronAdaptor.#2117
Tartarus0zm merged 1 commit intoapache:masterfrom
RocMarshal:AURON-2116

Conversation

@RocMarshal
Copy link
Contributor

Which issue does this PR close?

Closes #2116

Rationale for this change

fix the input stream leaking.

What changes are included in this PR?

Make libInputStream auto-close when creating in FlinkAuronAdaptor.java

Are there any user-facing changes?

N.A

How was this patch tested?

N.A

@github-actions github-actions bot added the flink label Mar 25, 2026
@RocMarshal RocMarshal closed this Mar 26, 2026
@RocMarshal RocMarshal reopened this Mar 26, 2026
@RocMarshal
Copy link
Contributor Author

Hi, @Tartarus0zm
Could you help take a look this minor path ?
Thank you very much

@Tartarus0zm
Copy link
Contributor

Hi, @Tartarus0zm Could you help take a look this minor path ? Thank you very much

@RocMarshal thanks for your contribution!
LGTM, SparkAuronAdaptor should be able to use the same optimization

…link/Spark AuronAdaptor.

Signed-off-by: Yuepeng Pan <panyuepeng@apache.org>
@github-actions github-actions bot added the spark label Mar 26, 2026
@RocMarshal RocMarshal changed the title [AURON #2116] Make libInputStream auto-close when creating in FlinkAuronAdaptor.java [AURON #2116] Make libInputStream auto-close when creating in Flink/Spark AuronAdaptor. Mar 26, 2026
@RocMarshal
Copy link
Contributor Author

Hi, @Tartarus0zm Could you help take a look this minor path ? Thank you very much

@RocMarshal thanks for your contribution! LGTM, SparkAuronAdaptor should be able to use the same optimization

Thanks @Tartarus0zm .

I updated the corresponding lines in spark extension.

Copy link
Contributor

@ShreyeshArangath ShreyeshArangath left a comment

Choose a reason for hiding this comment

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

Thanks for adding this! LGTM

Copy link
Contributor

@Tartarus0zm Tartarus0zm left a comment

Choose a reason for hiding this comment

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

LGTM

@Tartarus0zm Tartarus0zm merged commit 2b77f1e into apache:master Mar 26, 2026
124 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make libInputStream auto-close when creating in Flink/Spark AuronAdaptor.

3 participants