Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

investigate Context's syscalls usage #1366

Closed
Geal opened this issue Jul 6, 2022 · 2 comments
Closed

investigate Context's syscalls usage #1366

Geal opened this issue Jul 6, 2022 · 2 comments
Assignees

Comments

@Geal
Copy link
Contributor

Geal commented Jul 6, 2022

creating a new Context object surprisingly implies doing a file syscall:
Screenshot from 2022-07-06 16-37-14

This is apparently due to DashMap looking up the available resources in the cgroup at creation:
https://github.com/xacrimon/dashmap/blob/459db7ac6f3d0b46f507cb724dd9bb0ce389f4ae/src/lib.rs#L55

I opened xacrimon/dashmap#219 to see if this could be memoized. If not we could do it inside the router

@Geal
Copy link
Contributor Author

Geal commented Aug 11, 2022

this will be fixed once xacrimon/dashmap#222 is merged and published

@Geal
Copy link
Contributor Author

Geal commented Aug 31, 2022

closing this because DashMap 5.4.0 was released with a fix for that and added to the router in 0151285

@Geal Geal closed this as completed Aug 31, 2022
@abernix abernix added this to the v1.0.0-alpha.1 milestone Aug 31, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants