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

Replace int with string on thread monitor stat collection #310

Merged
merged 2 commits into from
Jun 28, 2022

Conversation

leoparente
Copy link
Collaborator

No description provided.

@leoparente leoparente requested a review from weyrick June 27, 2022 18:46
@leoparente leoparente self-assigned this Jun 27, 2022
@leoparente
Copy link
Collaborator Author

leoparente commented Jun 27, 2022

thread_stat.ignore(' '); was causing different behaviors, and stat has string and int values. So the best approach is to read everything as string and then convert the necessary values

sample stats:

98 (cat) R 18 198 18 34817 198 4194304 78 0 0 0 0 0 0 0 20 0 1 0 1299013 4505600 185 18446744073709551615 4194304 4238828 140735373488928 0 0 0 0 0 0 0 0 0 17 6 0 0 0 0 0 6339656 6341408 19660800 140735373490557 140735373490584 140735373490584 140735373492203 0

@leoparente leoparente linked an issue Jun 27, 2022 that may be closed by this pull request
@leoparente leoparente merged commit fbcfbb8 into develop Jun 28, 2022
@leoparente leoparente deleted the bugfix/thread-monitor-centos7 branch June 28, 2022 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ThreadMonitor failing on CentOS7
2 participants