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.

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

Jump to bottom

Default native reports to CSV #20733

Closed
galderz opened this issue Oct 13, 2021 · 1 comment · Fixed by #24268
Closed

Default native reports to CSV #20733

galderz opened this issue Oct 13, 2021 · 1 comment · Fixed by #24268
Assignees
@zakkak
Labels
area/native-image kind/enhancement New feature or request
Milestone
2.7.6.Final

Comments

@galderz
Copy link
Member

galderz commented Oct 13, 2021

Description

When implemented, integrate oracle/graal#3843 into Quarkus.

When a user sets -Dquarkus.native.enable-reports, Quarkus should instruct GraalVM to produce CSV call trees by default.

Quarkus should not offer a specific property for switching to a text format call tree. The user can always pass in -Dquarkus.native.additional-build-args=+-H:+PrintAnalysisCallTreeType=text if they really want the text format.

FYI @zakkak @Sanne

Implementation ideas

No response
@galderz galderz added the kind/enhancement New feature or request label Oct 13, 2021
@galderz galderz mentioned this issue Mar 2, 2022
Merged
zakkak added a commit to zakkak/quarkus that referenced this issue Mar 11, 2022
@zakkak
Use -H:PrintAnalysisCallTreeType=CSV to generate csv call tree files
8ee9946 
Fixes quarkusio#20733
@zakkak zakkak mentioned this issue Mar 11, 2022
Merged
zakkak added a commit to zakkak/quarkus that referenced this issue Mar 11, 2022
@zakkak
Use -H:PrintAnalysisCallTreeType=CSV to generate csv call tree files
c7d99ed 
Fixes quarkusio#20733
@galderz galderz assigned zakkak and unassigned galderz Mar 15, 2022
@galderz
Copy link
Member Author

galderz commented Mar 15, 2022

@zakkak Assigned this to you since you've worked on it.

zakkak added a commit to zakkak/quarkus that referenced this issue Mar 16, 2022
@zakkak
Use -H:PrintAnalysisCallTreeType=CSV to generate csv call tree files
b872464 
Fixes quarkusio#20733
zakkak added a commit to zakkak/quarkus that referenced this issue Mar 16, 2022
@zakkak
Use -H:PrintAnalysisCallTreeType=CSV to generate csv call tree files
4db18bc 
Fixes quarkusio#20733
@quarkus-bot quarkus-bot bot added this to the 2.8 - main milestone Mar 18, 2022
@gsmet gsmet modified the milestones: 2.8.0.CR1, 2.7.6.Final May 5, 2022
gsmet pushed a commit to gsmet/quarkus that referenced this issue May 5, 2022
@zakkak @gsmet
Use -H:PrintAnalysisCallTreeType=CSV to generate csv call tree files
69896e2 
Fixes quarkusio#20733

(cherry picked from commit 4db18bc)
gsmet pushed a commit to gsmet/quarkus that referenced this issue May 5, 2022
@zakkak @gsmet
Use -H:PrintAnalysisCallTreeType=CSV to generate csv call tree files
57f2e83 
Fixes quarkusio#20733

(cherry picked from commit 4db18bc)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zakkak zakkak

Labels
area/native-image kind/enhancement New feature or request
Projects
None yet
Milestone
2.7.6.Final
Development

Successfully merging a pull request may close this issue.

Use -H:PrintAnalysisCallTreeType=CSV to generate csv call tree files zakkak/quarkus
4 participants
@Sanne @galderz @gsmet @zakkak