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

Native Reference Guide: Instructions to get a thread dump #23837

Merged
merged 5 commits into from
Feb 22, 2022

Conversation

zakkak
Copy link
Contributor

@zakkak zakkak commented Feb 21, 2022

No description provided.

@gsmet
Copy link
Member

gsmet commented Feb 21, 2022

A thread dumb is someone not knowing anything about threads?

I'll go fix the PR title :).

@gsmet gsmet changed the title Native Reference Guide: Instructions to get a thread dumb Native Reference Guide: Instructions to get a thread dump Feb 21, 2022
gsmet
gsmet previously requested changes Feb 21, 2022
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Added a few minor comments.

docs/src/main/asciidoc/native-reference.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/native-reference.adoc Outdated Show resolved Hide resolved
@zakkak zakkak force-pushed the native-thread-backtraces branch from 4a5bf38 to 45cf86c Compare February 21, 2022 13:54
@zakkak zakkak force-pushed the native-thread-backtraces branch from 45cf86c to a50355e Compare February 22, 2022 09:55
@zakkak zakkak requested a review from gsmet February 22, 2022 11:34
@geoand geoand dismissed gsmet’s stale review February 22, 2022 15:46

comments addressed

@galderz galderz merged commit 41ef51e into quarkusio:main Feb 22, 2022
@quarkus-bot quarkus-bot bot added this to the 2.8 - main milestone Feb 22, 2022
@gsmet gsmet modified the milestones: 2.8 - main, 2.7.4.Final Mar 7, 2022
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