Skip to content

Conversation

@Jongy
Copy link
Contributor

@Jongy Jongy commented Jan 8, 2022

Description

AP itself is highly configurable. This option will allow trying out different AP configurations, even if they are not yet supported by gProfiler's interface.

How Has This Been Tested?

Added AP option dot and saw that Java class names are now using . instead of /.

@Jongy
Copy link
Contributor Author

Jongy commented Jan 8, 2022

In another PR I will organize the tests so we have 1 site creating JavaProfiler and we don't need to make the same change repeatedly in all sites.

@Jongy Jongy force-pushed the java-arbitrary-ap-opts branch from a84575f to ae98eac Compare January 17, 2022 12:07
@Jongy Jongy requested a review from DanielShaulov January 17, 2022 12:11
DanielShaulov
DanielShaulov previously approved these changes Jan 17, 2022
@Jongy Jongy merged commit 866e7c7 into master Jan 17, 2022
@Jongy Jongy deleted the java-arbitrary-ap-opts branch January 17, 2022 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants