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-include j.c.util.ArrayBlockingQueue/WhiteBox #1350

Merged
merged 1 commit into from
Sep 20, 2019

Conversation

theresa-m
Copy link
Contributor

@theresa-m theresa-m commented Sep 17, 2019

Excluded for issue: eclipse-openj9/openj9#5988
Fix for issue: eclipse-openj9/openj9#7104

Signed-off-by: Theresa Mammarella Theresa.T.Mammarella@ibm.com

@theresa-m
Copy link
Contributor Author

eclipse-openj9/openj9#7104 has been merged, this is ready to be reincluded.

Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

thanks @theresa-m !

Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

Does the fix also apply to 11+ platforms, if so, can you also update the other excludes, I find it in the jdk13 exclude file: https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/openjdk/ProblemList_openjdk13-openj9.txt#L309 etc

Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

Will hold off merge to get the update to the jdk13 exclude file if applicable

Signed-off-by: Theresa Mammarella <Theresa.T.Mammarella@ibm.com>
@theresa-m
Copy link
Contributor Author

Yes it applies to 13 as well. updated the pr.

@smlambert smlambert merged commit c91c600 into adoptium:master Sep 20, 2019
pshipton added a commit to pshipton/openjdk-tests that referenced this pull request Dec 20, 2022
It was unexcluded via adoptium#1350
but seems to have been accidently re-excluded via
adoptium#1348

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
pshipton added a commit to pshipton/openjdk-tests that referenced this pull request Dec 20, 2022
It was unexcluded via adoptium#1350
but seems to have been accidentally re-excluded via
adoptium#1348

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
smlambert pushed a commit that referenced this pull request Dec 20, 2022
It was unexcluded via #1350
but seems to have been accidentally re-excluded via
#1348

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants