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

HUGE off-heap memory leak and heap memory leak when using quick-js #61

Open
sandtechnology opened this issue Feb 17, 2022 · 4 comments
Open

Comments

@sandtechnology
Copy link

sandtechnology commented Feb 17, 2022

Not sure if the issue from quick-js or your side, but I post them to remind other server owner when troubleshooting.

Heap memory leak by mat report:

image
image

Merged path2gc shows it was referred by JNI Gobal ref:

image

Also off-heap having LARGE leak about 50GB+, jmap shows there are lots of JNI refs

image
image
image

@sandtechnology
Copy link
Author

sandtechnology commented Feb 17, 2022

And seems is the bug from quack which introduced by Javascript-Expansion 2.x, so I will try to use 1.x to see if it fixed it.

@sandtechnology sandtechnology changed the title HUGE off-heap memory leak and heap memory leak when using quick-js HUGE off-heap memory leak and heap memory leak when using 2.x version Feb 17, 2022
@sandtechnology sandtechnology changed the title HUGE off-heap memory leak and heap memory leak when using 2.x version HUGE off-heap memory leak and heap memory leak when using quick-js Feb 17, 2022
@partydev
Copy link

partydev commented Jun 3, 2022

Confirmed this with our testing. We were leaking 650mb/s at one point. Removing QuickJS solved it. We never had a heap memory leak, only native.

@darbyjack
Copy link
Member

@sandtechnology I don't know a whole lot of c++ but I saw there is a fork of quack that seems better maintained. If you understand it, can you check this fork to see if it looks like it fixes the problem? https://github.com/uk-taniyama/quack

@darbyjack
Copy link
Member

darbyjack commented Jun 3, 2022

I just implemented another possible engine to use called RhinoJS. I'm not entirely sure what all it offers but I tested basic script evaluations which seemed to work fine. I have a copy of the jar in this open PR #65

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

No branches or pull requests

3 participants