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

add profile task query segment list and analyzation #29

Merged
merged 6 commits into from Jan 31, 2020
Merged

add profile task query segment list and analyzation #29

merged 6 commits into from Jan 31, 2020

Conversation

mrproliu
Copy link
Contributor

No description provided.

@mrproliu mrproliu changed the title add query segment list by profile task add profile task query segment list and analyzation Jan 20, 2020
@wu-sheng wu-sheng added this to the 7.0.0 milestone Jan 31, 2020
@wu-sheng wu-sheng added the enhancement New feature or request label Jan 31, 2020
profile.graphqls Outdated Show resolved Hide resolved
profile.graphqls Outdated Show resolved Hide resolved
profile.graphqls Outdated Show resolved Hide resolved
profile.graphqls Show resolved Hide resolved
@wu-sheng
Copy link
Member

After this merged, recommend you to send a separate pull request and just implement the shell of this protocol, make compiling and boot successfully. Because otherwise, it may block other PRs.

@mrproliu
Copy link
Contributor Author

After this merged, recommend you to send a separate pull request and just implement the shell of this protocol, make compiling and boot successfully. Because otherwise, it may block other PRs.

Sure.

profile.graphqls Outdated
# total dump count
count: Int!
# max recursion invoke count
maxRecursionCount: Int!
Copy link
Member

Choose a reason for hiding this comment

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

Don't recommend use this. Stack depth is limited in the agent. We could say it is safe here.

profile.graphqls Outdated
@@ -84,6 +84,28 @@ type ProfileTask {
logs: [ProfileTaskLog!]!
}

# Profile thread stack anayze tree
type ThreadStackAnalyzation {
Copy link
Member

@wu-sheng wu-sheng Jan 31, 2020

Choose a reason for hiding this comment

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

ProfileStackElement

@wu-sheng
Copy link
Member

Please provide the implementation and update the submodule in the main repo.

@wu-sheng wu-sheng merged commit 7db60e3 into apache:master Jan 31, 2020
@mrproliu mrproliu deleted the profile_task_segment_query branch January 31, 2020 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants