-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Out of Memory in deoptimizeThisOnEventAtPath since rollup 2.49.0 #4089
Comments
I will have a look, could be a bug |
Thanks for spotting, fix here: #4091 |
I'm still seeing stack overflow errors in our builds that started with
Link to our CI run if that's helpful: |
Not really, a reproduction would be helpful. Otherwise I need to manually sift through your code which could take ages. |
So what is likely happening is that there is an infinite loop somewhere that spans several functions. I need to find out the code sample that triggers it. |
@lukeapage would it make sense to reopen this issue? To help keep track of "ongoing" issues? 😄 |
Without a reproduction there is nothing we can do, we do not even know if it is just an error on user side i.e. wrong Rollup version. |
So to repeat THIS ERROR WILL NOT BE FIXED WITHOUT YOUR HELP. |
@lukeapage thank you for that helpful clarification. 👍 |
Understood. Two of us on our team each spent a whole day trying to narrow down what's causing this but didn't really end up anywhere -- so we had to pin Rollup at Our application is quite large with lots of 3rd party dependencies. Excluding any one of several parts of the application from the Rollup build stopped the overflow errors, but we weren't able to narrow things down to a single block of code. We were pretty confused about why excluding different parts on their own would solve the problem. For example, if the build had We are pretty sure it's not a user-side error with the wrong Rollup version though... |
From what you describe, it could also be that you just hit a "regular" memory limit. Does
have any positive effect for you? Note that you can use very large numbers here that exceed actual physical memory without issues. Otherwise, if the code is public you can point me to what repo I have to check out and give me instructions how to build it and I can have a go. |
Expected Behavior
Built with no error.
src/marked.js
is copied frommonaco-editor/esm/vs/base/common/marked/marked.js
which is part of monaco-editor.Tried enlarging node memory size by using
--max-old-space-size=10240
, still crashed.Actual Behavior
Rollup run infinitely and finally crashed because of allocation failure.
The text was updated successfully, but these errors were encountered: