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

CoordinatorTests testLeaderDisconnectionWithDisconnectEventDetectedQuickly not enough memory to allocate buffers #48742

Closed
albertzaharovits opened this issue Oct 31, 2019 · 1 comment · Fixed by #48751
Closed

CoordinatorTests testLeaderDisconnectionWithDisconnectEventDetectedQuickly not enough memory to allocate buffers #48742

albertzaharovits opened this issue Oct 31, 2019 · 1 comment · Fixed by #48751
Assignees
@original-brownbear
Labels
:Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >test-failure Triaged test failures from CI

Comments

@albertzaharovits
Copy link
Contributor

The following fails reproducible:

./gradlew ':server:test' --tests "org.elasticsearch.cluster.coordination.CoordinatorTests.testLeaderDisconnectionWithDisconnectEventDetectedQuickly" -Dtests.seed=E0335BED6EE9F783 -Dtests.security.manager=true -Dtests.locale=es-UY -Dtests.timezone=MIT -Dcompiler.java=12 -Druntime.java=11

with

        Caused by:
        java.lang.OutOfMemoryError: Java heap space
            at com.carrotsearch.hppc.LongObjectHashMap.allocateBuffers(LongObjectHashMap.java:1102)
            at com.carrotsearch.hppc.LongObjectHashMap.allocateThenInsertThenRehash(LongObjectHashMap.java:1136)
            at com.carrotsearch.hppc.LongObjectHashMap.indexInsert(LongObjectHashMap.java:520)
            at org.elasticsearch.cluster.coordination.LinearizabilityChecker$Cache.addSmall(LinearizabilityChecker.java:526)
            at org.elasticsearch.cluster.coordination.LinearizabilityChecker$Cache.addInternal(LinearizabilityChecker.java:502)
            at org.elasticsearch.cluster.coordination.LinearizabilityChecker$Cache.add(LinearizabilityChecker.java:496)
            at org.elasticsearch.cluster.coordination.LinearizabilityChecker.isLinearizable(LinearizabilityChecker.java:273)
            at org.elasticsearch.cluster.coordination.LinearizabilityChecker.lambda$isLinearizable$1(LinearizabilityChecker.java:248)
            at org.elasticsearch.cluster.coordination.LinearizabilityChecker$$Lambda$952/0x000000080053cc40.test(Unknown Source)
            ... 15 more

in https://gradle-enterprise.elastic.co/s/qb7pyn3n5mdw2

Looks like 60% of the heap is taken by long arrays.

 
@albertzaharovits albertzaharovits added >test-failure Triaged test failures from CI :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. labels Oct 31, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Cluster Coordination)

@original-brownbear original-brownbear self-assigned this Oct 31, 2019
@original-brownbear original-brownbear mentioned this issue Oct 31, 2019
Merged
@original-brownbear original-brownbear mentioned this issue Nov 4, 2019
Merged
original-brownbear added a commit that referenced this issue Nov 4, 2019
@original-brownbear
Bound Linearizability Check in CoordinatorTests (#48751) (#48853)
d83e374 
Same as #44444 but for the coordinator tests.
Closes #48742
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@original-brownbear original-brownbear

Labels
:Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bound Linearizability Check in CoordinatorTests original-brownbear/elasticsearch
3 participants
@albertzaharovits @original-brownbear @elasticmachine