-
Notifications
You must be signed in to change notification settings - Fork 24.6k
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
index slowlog settings override other indices settings #42432
Labels
Comments
Pinging @elastic/es-core-infra |
pgomulka
pushed a commit
that referenced
this issue
Oct 17, 2019
SlowLog instances should not share the same underlying logger, as it would cause different indexes override each other levels. When creating underlying logger, unique per index identifier should be used. Name + IndexSettings.UUID Closes #42432
pgomulka
pushed a commit
to pgomulka/elasticsearch
that referenced
this issue
Oct 17, 2019
…47234) SlowLog instances should not share the same underlying logger, as it would cause different indexes override each other levels. When creating underlying logger, unique per index identifier should be used. Name + IndexSettings.UUID Closes elastic#42432
pgomulka
pushed a commit
to pgomulka/elasticsearch
that referenced
this issue
Oct 17, 2019
…47234) SlowLog instances should not share the same underlying logger, as it would cause different indexes override each other levels. When creating underlying logger, unique per index identifier should be used. Name + IndexSettings.UUID Closes elastic#42432
pgomulka
added a commit
that referenced
this issue
Oct 17, 2019
…KPORT(#47234) (#48176) * Slow log must use separate underlying logger for each index (#47234) SlowLog instances should not share the same underlying logger, as it would cause different indexes override each other levels. When creating underlying logger, unique per index identifier should be used. Name + IndexSettings.UUID Closes #42432
pgomulka
added a commit
that referenced
this issue
Oct 17, 2019
…KPORT(#47234) (#48177) * Slow log must use separate underlying logger for each index (#47234) SlowLog instances should not share the same underlying logger, as it would cause different indexes override each other levels. When creating underlying logger, unique per index identifier should be used. Name + IndexSettings.UUID Closes #42432
This was referenced Feb 3, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Elasticsearch version : Tested with 6.6.1 and master
Description of the problem including expected versus actual behavior:
Slowlog settings are supposed to be per index. However, setting the slowlog settings for an index, override the respective settings in all other indices, even when these other settings have been previously configured.
Steps to reproduce:
test_1
and setindex.search.slowlog.level: warn
andindex.search.slowlog.threshold.query.trace
to 0.Get the settings to verify it is set correctly
should return
and verify nothing is printed in the
*.index_search_slowlog.log
test_2
and setindex.search.slowlog.level: trace
andindex.search.slowlog.threshold.query.trace
to 0.Get the settings to verify it is set correctly
should return
Verify also that the settings for
test_1
have not been changed ( at least that's reported ):should still return
test_1
againObserve log lines such as
in the
*.index_search_slowlog.log
The text was updated successfully, but these errors were encountered: