-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Implement fast dump #2047
Comments
All you need to do is to add the dependency of ‘fast-dump’, 'DumpWrapper' will automatically do the rest. Support Android 5~11. Signed-off-by: xueqiushi <xueqiushi@kuaishou.com>
All you need to do is to add the dependency of ‘fast-dump’, 'DumpWrapper' will automatically do the rest. Support Android 5~11. Signed-off-by: xueqiushi <xueqiushi@kuaishou.com>
Sorry for the late submission. Recently, we are busy with Spring Festival activities. Every Chinese Internet company is the busiest during the months around the Spring Festival. Later, I discussed with the company's legal team to obtain permission to release the code. Please help to review my code
|
All you need to do is to add the dependency of ‘fast-dump’, 'DumpWrapper' will automatically do the rest. Support Android 5~11. Signed-off-by: xueqiushi <xueqiushi@kuaishou.com>
Cool!
Oops I didn't realized that was done here: #2121 |
This will be released as part of KOOM now that the API is pluggable. #2121 (comment) |
Today, heap dump is slow and blocks the VM. https://github.com/KwaiAppTeam/KOOM showed that it can be done as fork and dump for a number of archectures, which is much faster.
Proposal: create a new set of modules for heap dumping. That way we separate the default android impl, the native fast impl amd the jvm impl. These modules should stand on their own, e.g. if someone just wants heap dumping.
The native libs for fast dump should probably not be included for a few default for a few releases. Some people want to keep binary size small.
The text was updated successfully, but these errors were encountered: