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

Add option to increase heap maximum in OOM notification #700

Closed
inikishev opened this issue Apr 18, 2022 · 14 comments · Fixed by #718
Closed

Add option to increase heap maximum in OOM notification #700

inikishev opened this issue Apr 18, 2022 · 14 comments · Fixed by #718
Assignees
Labels
bug Something isn't working good first issue Good for newcomers
Milestone

Comments

@inikishev
Copy link

I need to give the thing more memory.
It says "go to xml.server.vmargs and add -Xmx1G.
I search for xml.server.vmargs in settings and press Edit in settings.json
image
thats what the file looks like
image
where do i put the " -Xmx1G."

@angelozerr
Copy link
Contributor

Are you in untrusted workspace?

Have you tried to write the settings at hand?

@inikishev
Copy link
Author

Are you in untrusted workspace?

Have you tried to write the settings at hand?

sorry I am not sure what that means, how do I know if I am in untrusted workspace and how do I try to to write the settings at hand? I sometimes notice there is a blue notification saying something about untrusted workspace, but it doesn't appear when I try to edit this setting.

@angelozerr
Copy link
Contributor

@inikishev
Copy link
Author

inikishev commented Apr 19, 2022

I suggest that you read https://code.visualstudio.com/blogs/2021/07/06/workspace-trust

i disabled the trust in accordance to the article and nothing changed, I still don't know where to put -Xmx1G.

@fbricon
Copy link
Collaborator

fbricon commented Apr 19, 2022

simply type "xml.server.vmargs" in the User settings.json file

@inikishev
Copy link
Author

simply type "xml.server.vmargs" in the User settings.json file

It doesn't like it no matter how i put it
image
image
image

@fbricon
Copy link
Collaborator

fbricon commented Apr 19, 2022

looks like vscode-xml is not installed/active on your machine

@fbricon
Copy link
Collaborator

fbricon commented Apr 19, 2022

Ctrl+Space after xml.server.vmargs.
Eventually it should look like "xml.server.vmargs":"-Xmx1G", once you configure it

@inikishev
Copy link
Author

looks like vscode-xml is not installed/active on your machine

it definitely is intalled because i am currently using it
image

@inikishev
Copy link
Author

inikishev commented Apr 19, 2022

Ctrl+Space after xml.server.vmargs. Eventually it should look like "xml.server.vmargs":"-Xmx1G", once you configure it

where do i put it in the file?

@fbricon
Copy link
Collaborator

fbricon commented Apr 19, 2022

towards the end of the settings.json file for instance, before the closing bracket.

@angelozerr
Copy link
Contributor

If you go to your second line of your settings.json and you type xml, you should see:

image

if you continue to type xml.ser, you should see:

image

@angelozerr
Copy link
Contributor

i disabled the trust in accordance to the article and nothing changed, I still don't know where to put -Xmx1G.

Your workspace should be in trust mode.

@fbricon @rgrunber to avoid this problem, I think the popup which display the out of memory problem should provide a new button Apply suggestion to update the setting xml.server.vmargs with the proper value. What do you think about that?

@rgrunber
Copy link
Member

rgrunber commented Jun 2, 2022

We seem to do that in vscode-java. redhat-developer/vscode-java#2085 . One of the options is "Configure" which appears to double the given XmX value. We should do the same.

https://github.com/redhat-developer/vscode-java/blob/master/src/extension.ts#L87

@rgrunber rgrunber added bug Something isn't working good first issue Good for newcomers labels Jun 3, 2022
@rgrunber rgrunber changed the title Server VM Arguments Can be accessed through xml.server.vmargs how do I access them? Add option to increase heap maximum in OOM notification Jun 3, 2022
datho7561 added a commit to datho7561/vscode-xml that referenced this issue Jun 8, 2022
Copied from vscode-java.

Closes redhat-developer#700

Signed-off-by: David Thompson <[email protected]>
datho7561 added a commit to datho7561/vscode-xml that referenced this issue Jun 10, 2022
Copied from vscode-java.

Closes redhat-developer#700

Signed-off-by: David Thompson <[email protected]>
@rgrunber rgrunber added this to the 0.21.0 milestone Jun 10, 2022
datho7561 added a commit to datho7561/vscode-xml that referenced this issue Jun 10, 2022
Copied from vscode-java.

Closes redhat-developer#700

Signed-off-by: David Thompson <[email protected]>
datho7561 added a commit that referenced this issue Jun 10, 2022
Copied from vscode-java.

Closes #700

Signed-off-by: David Thompson <[email protected]>
@angelozerr angelozerr added enhancement New feature or request bug Something isn't working and removed bug Something isn't working enhancement New feature or request labels Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants