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

Return index set wildcard instead of individual indices if index list is too long #4078

Merged
merged 1 commit into from Aug 11, 2017

Conversation

Projects
None yet
2 participants
@joschi
Contributor

joschi commented Aug 11, 2017

Refs #4062
Fixes #4054
(cherry picked from commit 3dc4ca1)

Return index set wildcard instead of individual indices if index list…
… is too long (#4062)

* Return write index alias instead of individual indices in Searches#determineAffectedIndices()

If the number of indices determined in `Searches#determineAffectedIndices()` is larger than
`Searches.MAX_INDICES_PER_QUERY`, return the covering write index aliases of the respective
index sets instead of the individual index names.

Fixes #4054

* Use index wildcard instead of write index alias (duh!)

* Use correct indices in Searches#search(SearchesConfig)

* Reduce number of indices in Searches#fieldStats() if necessary

(cherry picked from commit 3dc4ca1)

@joschi joschi added this to the 2.3.1 milestone Aug 11, 2017

@bernd bernd self-assigned this Aug 11, 2017

@bernd

bernd approved these changes Aug 11, 2017

LGTM 👍

@bernd bernd merged commit d148e79 into 2.3 Aug 11, 2017

0 of 4 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 1863 is running
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
graylog-project/pr Jenkins build graylog-project-pr-snapshot 392 is running
Details

@bernd bernd deleted the pr-4062-2.3 branch Aug 11, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment