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

AsyncTask timings #6166

Open
dktapps opened this issue Nov 18, 2023 · 1 comment
Open

AsyncTask timings #6166

dktapps opened this issue Nov 18, 2023 · 1 comment
Labels
Category: Core Related to internal functionality Type: Enhancement Contributes features or other improvements to PocketMine-MP

Comments

@dktapps
Copy link
Member

dktapps commented Nov 18, 2023

Description

It should be possible to add separate tables on timings to insert timings from async workers.
These would be separate from Full Server Tick, so they wouldn't be counted towards main server load.

This would give a more complete picture of server performance.

Alternative methods
@dktapps dktapps added Category: Core Related to internal functionality Type: Enhancement Contributes features or other improvements to PocketMine-MP labels Nov 18, 2023
@dktapps
Copy link
Member Author

dktapps commented Feb 13, 2024

Turns out this is more complicated than I expected, because the timings record IDs from different threads may collide. We'd need some way to namespace them.

@dktapps dktapps mentioned this issue Apr 19, 2024
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Category: Core Related to internal functionality Type: Enhancement Contributes features or other improvements to PocketMine-MP
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dktapps