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

Re-enable JVMTI test WaitNotifySuspendedVThreadTest #4558

Merged
merged 1 commit into from
May 3, 2023

Conversation

dipak-bagadiya
Copy link
Contributor

@dipak-bagadiya dipak-bagadiya commented May 3, 2023

Test has been fixed by eclipse-openj9/openj9#16943

Signed-off-by: Dipak Bagadiya dipak.bagadiya@ibm.com

@dipak-bagadiya dipak-bagadiya marked this pull request as ready for review May 3, 2023 14:41
@babsingh
Copy link
Contributor

babsingh commented May 3, 2023

Grammar nitpick (The test has been ...):

The test has been fixed by eclipse-openj9/openj9#16943.

Test has been fixed by eclipse-openj9/openj9#16943.

Signed-off-by: Dipak Bagadiya dipak.bagadiya@ibm.com
@dipak-bagadiya dipak-bagadiya deleted the test_fix_suspend_resume branch May 3, 2023 15:11
@dipak-bagadiya dipak-bagadiya restored the test_fix_suspend_resume branch May 3, 2023 15:12
@dipak-bagadiya dipak-bagadiya reopened this May 3, 2023
@dipak-bagadiya
Copy link
Contributor Author

@babsingh as indicated, the sentence has been altered.
Sorry, I accidentally closed the PR and then enabled it.

Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants