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.

Sign up for GitHub

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

Jump to bottom

Elasticsearch tests use to much memory when autoconfigured on my desktop/home server ;) #4831

Closed
JervenBolleman opened this issue Nov 1, 2023 · 1 comment
Closed

Elasticsearch tests use to much memory when autoconfigured on my desktop/home server ;) #4831

JervenBolleman opened this issue Nov 1, 2023 · 1 comment
Assignees
@JervenBolleman
Labels
🐞 bug issue is a bug 🔧 internal task a project workflow / setup task not directly impacting end users

Comments

@JervenBolleman
Copy link
Contributor

Current Behavior

Elasticsearch tests spin up a node asking for 32GB of heap space on my desktop.

Expected Behavior

Use about 1g or less for the unit tests

Steps To Reproduce

No response

Version

5.0.0-DEVELOP

Are you interested in contributing a solution yourself?

Yes

Anything else?

We can just set <ES_JAVA_OPTS>${java.sec.mgr}</ES_JAVA_OPTS> in the /core/sail/elasticsearch-store/pom.xml to limit the heap to 1gb. 1gb is the setting determined in the github actions.
@JervenBolleman JervenBolleman added the 🐞 bug issue is a bug label Nov 1, 2023
@JervenBolleman JervenBolleman self-assigned this Nov 1, 2023
JervenBolleman added a commit that referenced this issue Nov 3, 2023
@JervenBolleman
GH-4831 Elasticsearch store created by unit test uses max 1gb heap
53d9995 
Signed-off-by: Jerven Bolleman <jerven.bolleman@sib.swiss>
@JervenBolleman JervenBolleman mentioned this issue Nov 3, 2023
Merged
5 tasks
JervenBolleman added a commit to JervenBolleman/rdf4j that referenced this issue Nov 5, 2023
@JervenBolleman
eclipse-rdf4jGH-4831 Elasticsearch store created by unit test uses ma…
ef16a8a 
…x 1gb heap

Signed-off-by: Jerven Bolleman <jerven.bolleman@sib.swiss>
hmottestad added a commit that referenced this issue Nov 7, 2023
@hmottestad
GH-4831 Elasticsearch store created by unit test uses max 1gb heap (#…
be1d764 
…4832)
@hmottestad hmottestad added the 🔧 internal task a project workflow / setup task not directly impacting end users label Nov 7, 2023
@hmottestad hmottestad modified the milestone: 4.3.8 Nov 7, 2023
@hmottestad
Copy link
Contributor

I merged you PR, but maybe we should have the same fix on the main branch?

JervenBolleman added a commit that referenced this issue Nov 8, 2023
@JervenBolleman
GH-4831 Limit the elasticsearch node for unit tests to 1gb of Java heap
cb8a170
JervenBolleman added a commit that referenced this issue Nov 8, 2023
@JervenBolleman
GH-4831 Limit the elasticsearch node for unit tests to 1gb of Java heap
bf1735b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JervenBolleman JervenBolleman

Labels
🐞 bug issue is a bug 🔧 internal task a project workflow / setup task not directly impacting end users
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hmottestad @JervenBolleman