Skip to content

Separate hotspot_custom test for j9#5493

Merged
smlambert merged 1 commit intoadoptium:masterfrom
LongyuZhang:hotspotc1
Aug 13, 2024
Merged

Separate hotspot_custom test for j9#5493
smlambert merged 1 commit intoadoptium:masterfrom
LongyuZhang:hotspotc1

Conversation

@LongyuZhang
Copy link
Copy Markdown
Contributor

Separate hotspot_custom and hotspot_custom_j9 tests

Issue: #5491

@LongyuZhang
Copy link
Copy Markdown
Contributor Author

LongyuZhang commented Aug 7, 2024

Test Link:
**hotspot: hotspot_custom **

openj9: hotspot_custom_j9

  • with target serviceability/jvmti/GetSystemProperty/JvmtiGetSystemPropertyTest.java
    21: hyc_grinder 42629
    23: hyc_grinder 42630
  • no CUSTOM_TARGET using default:
    21: hyc_grinder 42628
    23: hyc_grinder 42631

@LongyuZhang LongyuZhang requested review from llxia and sophia-guo August 7, 2024 19:47
@sophia-guo sophia-guo dismissed their stale review August 13, 2024 19:01

no interested

Add hotspot_custom_j9 test

Signed-off-by: LongyuZhang <longyu.zhang@ibm.com>
@LongyuZhang LongyuZhang requested a review from smlambert August 13, 2024 21:01
Copy link
Copy Markdown
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.

As per the discussion this afternoon, approving, you may want to update openj9 docs regarding the nuance of the use of the hotspot_custom_j9 target.

@smlambert smlambert merged commit e6bb784 into adoptium:master Aug 13, 2024
llxia pushed a commit to llxia/aqa-tests that referenced this pull request Aug 14, 2024
Add hotspot_custom_j9 test

Signed-off-by: LongyuZhang <longyu.zhang@ibm.com>
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