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

Fix index set overview with closed indices #3930

Merged
merged 1 commit into from Jun 22, 2017
Merged

Conversation

@bernd
Copy link
Member

@bernd bernd commented Jun 22, 2017

The code was trying to modify an immutable map with details for closed indices.

Fixes the index overview page when closed indices are present.

2017-06-22 13:53:18,603 ERROR: org.graylog2.shared.rest.exceptionmappers.AnyExceptionClassMapper - Unhandled exception in REST resource
java.lang.UnsupportedOperationException: null
    at com.google.common.collect.ImmutableMap.put(ImmutableMap.java:473) ~[graylog.jar:?]
    at org.graylog2.rest.resources.system.indexer.IndexerOverviewResource.lambda$getIndexerOverview$1(IndexerOverviewResource.java:128) ~[graylog.jar:?]
    at java.lang.Iterable.forEach(Iterable.java:75) ~[?:1.8.0_131]
    at org.graylog2.rest.resources.system.indexer.IndexerOverviewResource.getIndexerOverview(IndexerOverviewResource.java:128) ~[graylog.jar:?]
    at org.graylog2.rest.resources.system.indexer.IndexerOverviewResource.index(IndexerOverviewResource.java:114) ~[graylog.jar:?]
The code was trying to modify an immutable map with details for closed
indices.
@bernd bernd added this to the 2.3.0 milestone Jun 22, 2017
@bernd bernd requested a review from dennisoelkers Jun 22, 2017
@dennisoelkers dennisoelkers merged commit 26aaf61 into master Jun 22, 2017
4 checks passed
4 checks passed
@garybot2
ci-web-linter Jenkins build graylog-pr-linter-check 1773 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@garybot2
graylog-project/pr Jenkins build graylog-project-pr-snapshot 270 has succeeded
Details
@dennisoelkers dennisoelkers deleted the fix-index-overview branch Jun 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants