Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

[hot-fix] increase data-node memory limits to 4Gi #1689

Merged
merged 3 commits into from
Nov 15, 2018

Conversation

hao1939
Copy link
Contributor

@hao1939 hao1939 commented Nov 9, 2018

Increase data-node memory limits to 4Gi.

@hao1939 hao1939 changed the title increate data-node memory limits to 4Gi [hot-fix]increate data-node memory limits to 4Gi Nov 9, 2018
@hao1939 hao1939 changed the title [hot-fix]increate data-node memory limits to 4Gi [hot-fix] increase data-node memory limits to 4Gi Nov 9, 2018
@coveralls
Copy link

coveralls commented Nov 9, 2018

Coverage Status

Coverage remained the same at 51.638% when pulling 8db01fa on hao/hotfix-datanode-memory-limit into b75ec9e on pai-0.8.y.

@@ -52,6 +52,8 @@ spec:
periodSeconds: 3
resources:
limits:
memory: "4Gi"
requests:
memory: "1Gi"
env:
- name: HADOOP_DATANODE_OPTS
Copy link
Member

@mzmssg mzmssg Nov 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to raise JVM memory opts? I think MT limitation is at JVM level.

Copy link
Contributor Author

@hao1939 hao1939 Nov 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure, since now we haven't found any java limit was hit.

Copy link
Member

@mzmssg mzmssg Nov 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#1658 For 175node screenshot, jave OOM is recorded. So we might need a larger JVM opts.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mzmssg ,
What about increase the max heap size to 2G?

@hao1939
Copy link
Contributor Author

hao1939 commented Nov 12, 2018

@mzmssg ,

Got you. If we want to guarantee, we should increase the reserved memory.

@hao1939 hao1939 force-pushed the hao/hotfix-datanode-memory-limit branch from ab64f6f to 8db01fa Compare November 13, 2018 05:46
@hao1939
Copy link
Contributor Author

hao1939 commented Nov 13, 2018

Hi @mzmssg , increase the reserved memory on worker to 16G.

@hao1939 hao1939 merged commit 321dfa7 into pai-0.8.y Nov 15, 2018
hao1939 added a commit that referenced this pull request Nov 15, 2018
* increate data-node memory limits to 4Gi

* increase data-node max java heapsize to 2G

* reserve more memory on PAI worker
@hao1939 hao1939 mentioned this pull request Nov 15, 2018
hao1939 added a commit that referenced this pull request Nov 16, 2018
* [hotfix] remove memory limits for k8s services and increase it for rm, zookeeper (#1619)

* increate etcd memory limits to 8Gi

* set the "requests" instead of "limits" for k8s services

* increase memory limits for rm

* increase memory limit for zookeeper

* add "requests" for rm

* fix typo

* [hot-fix] increase data-node memory limits to 4Gi (#1689)

* increate data-node memory limits to 4Gi

* increase data-node max java heapsize to 2G

* reserve more memory on PAI worker
@hao1939 hao1939 deleted the hao/hotfix-datanode-memory-limit branch December 6, 2018 09:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants