Skip to content

JVM Flags access API #149

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.

Sign up for GitHub

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

Merged
jbachorik merged 15 commits into from
Nov 12, 2024
Merged

JVM Flags access API #149

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
a59fc26
Introduce an API to access JVM flags directly
jbachorik Nov 5, 2024
5559c50
Try increasing retry limit for ObjectSampleDumpSmokeTest
jbachorik Nov 5, 2024
c71a738
Add more javadocs
jbachorik Nov 5, 2024
82fa8a1
Fix assumption
jbachorik Nov 5, 2024
619c5b7
Fix condition
jbachorik Nov 5, 2024
340884a
Zing does not support vmstructs, too
jbachorik Nov 5, 2024
6dd9ae7
Deal with the JVM flag implementation differences
jbachorik Nov 6, 2024
1ceb055
Decouple library initialization from profiler initialization
jbachorik Nov 6, 2024
bb4e9ea
success -> succeeded
jbachorik Nov 8, 2024
a5e686e
Update ddprof-test/src/test/java/com/datadoghq/profiler/JVMAccessTest…
jbachorik Nov 8, 2024
4c214f3
Update ddprof-test/src/test/java/com/datadoghq/profiler/JavaProfilerT…
jbachorik Nov 8, 2024
078b5f7
Use shared Libraries instance
jbachorik Nov 11, 2024
ae50b2a
Simplify JVMFlags API
jbachorik Nov 11, 2024
24a142f
Fix flag C++ calls
jbachorik Nov 11, 2024
4102eb3
More fixes
jbachorik Nov 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Zing does not support vmstructs, too
  • Loading branch information
@jbachorik
jbachorik committed Nov 8, 2024
commit 340884a31d7053cdc5e851177adae9addd00be55
2 changes: 1 addition & 1 deletion ddprof-test/src/test/java/com/datadoghq/profiler/JVMAccessTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ public class JVMAccessTest {

@BeforeAll
static void setUp() {
Assumptions.assumeFalse(Platform.isJ9()); // J9 does not support vmstructs
Assumptions.assumeFalse(Platform.isJ9() || Platform.isZing()); // J9 and Zing do not support vmstructs
jvmAccess = JVMAccess.getInstance();
}

Expand Down