-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Set minimum aggregation 'Size' input value to 1, because ES will return an error if you provide a size of 0. #8339
Conversation
743efcf
to
fa12b92
Compare
…rn an error if you provide a size of 0. - Display feedback if a Dashboard or Visualization is loaded that already violates this constraint. - Add ElasticsearchError class and isTermSizeZeroError helper.
fa12b92
to
4fefae1
Compare
LGTM |
1 similar comment
LGTM |
@@ -155,7 +159,17 @@ uiModules | |||
return searchSource.onResults().then(onResults); | |||
}).catch(notify.fatal); | |||
|
|||
searchSource.onError(notify.error).catch(notify.fatal); | |||
searchSource.onError(e => { | |||
if (isTermSizeZeroError(e)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would it be possible to include vis titles for unknown errors too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this idea, but it gets a little tricky given the implicit way our notification error messages are created. Let's address this as part of #8194
--------- **Commit 1:** Set minimum aggregation 'Size' input value to 1, because ES will return an error if you provide a size of 0. - Display feedback if a Dashboard or Visualization is loaded that already violates this constraint. - Add ElasticsearchError class and isTermSizeZeroError helper. * Original sha: 4fefae1 * Authored by CJ Cenizal <cj@cenizal.com> on 2016-09-19T19:02:54Z
--------- **Commit 1:** Set minimum aggregation 'Size' input value to 1, because ES will return an error if you provide a size of 0. - Display feedback if a Dashboard or Visualization is loaded that already violates this constraint. - Add ElasticsearchError class and isTermSizeZeroError helper. * Original sha: 4fefae1 * Authored by CJ Cenizal <cj@cenizal.com> on 2016-09-19T19:02:54Z
[backport] PR #8339 to 5.x - Set minimum aggregation 'Size' input value to 1, because ES will return an error if you provide a size of 0.
[backport] PR #8339 to 5.0 - Set minimum aggregation 'Size' input value to 1, because ES will return an error if you provide a size of 0.
--------- **Commit 1:** Set minimum aggregation 'Size' input value to 1, because ES will return an error if you provide a size of 0. - Display feedback if a Dashboard or Visualization is loaded that already violates this constraint. - Add ElasticsearchError class and isTermSizeZeroError helper. * Original sha: cd604dea0c6ffcb454c96e3ded5c00690d9af33c [formerly 4fefae1] * Authored by CJ Cenizal <cj@cenizal.com> on 2016-09-19T19:02:54Z Former-commit-id: 0ca20ac
[backport] PR elastic#8339 to 5.x - Set minimum aggregation 'Size' input value to 1, because ES will return an error if you provide a size of 0. Former-commit-id: a4df947
Addresses #8330
Changes