Skip to content

Conversation

@chatman
Copy link
Contributor

@chatman chatman commented Feb 17, 2025

Using a pool memory manager was causing crash with different threads. Modified a test to run parallely sometimes.

Using a pool memory manager was causing crash with different threads. Modified a test to run parallely sometimes.

Co-authored-by: Vivek Narang <[email protected]>
@copy-pr-bot
Copy link

copy-pr-bot bot commented Feb 17, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@chatman chatman changed the title Fix for a crash situation when using different threads (#680) Java: Fix for a crash situation when using different threads (#680) Feb 17, 2025
@chatman chatman changed the base branch from branch-25.04 to branch-25.02 February 17, 2025 06:24
@chatman
Copy link
Contributor Author

chatman commented Feb 17, 2025

Branch 25.04 doesn't have latest Java fixes that 25.02 has. Hence, unable to raise against that branch.

@rhdong rhdong added bug Something isn't working non-breaking Introduces a non-breaking change Java labels Feb 17, 2025
Copy link
Member

@rhdong rhdong left a comment

Choose a reason for hiding this comment

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

LGTM

@rhdong
Copy link
Member

rhdong commented Feb 17, 2025

/ok to test

@rhdong
Copy link
Member

rhdong commented Feb 17, 2025

Hi @cjnolet , would you like this PR could be squashed with #695 ?

@cjnolet cjnolet changed the base branch from branch-25.02 to branch-25.06 April 25, 2025 15:11
@chatman
Copy link
Contributor Author

chatman commented May 21, 2025

This is outdated due to test fixes done in other PRs.

@chatman chatman closed this May 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Java non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants