Skip to content

[6.x] Enable KQL support for TSVB (#26006)#29828

Merged
markov00 merged 3 commits intoelastic:6.xfrom
markov00:backport/6.x/pr-26006
Feb 5, 2019
Merged

[6.x] Enable KQL support for TSVB (#26006)#29828
markov00 merged 3 commits intoelastic:6.xfrom
markov00:backport/6.x/pr-26006

Conversation

@markov00
Copy link
Copy Markdown
Contributor

@markov00 markov00 commented Feb 1, 2019

Backports the following commits to 6.x:

* Use buildEsQuery for table, series and annotations

* Fix query test.

Using the buildEsQuery changed a bit the order of the must.bool array on the query

* Remove console.log

* Remove console.error and comment

* Fix wrong merge of PR elastic#26510

* Fix default/empty index_pattern

When the user save the visualization without configuring
manually an index_pattern, a default empty string is used
instead of the default pattern. This leads to an empty
visualization after the refactoring done in elastic#24832.
Now it will update the index_pattern field with the default
index pattern in visualize editor and on dashboard.

* Remove unnecessary wrapping query in an array

* Enable query bar on tsvb

* Remove unnecessary setDefaultIndexPattern

After fixing the null index pattern issue in kql
there is no need to use set the default index pattern
before rendering.

* fix(tsvb-server): Ignore query bar search on ignore_global_filters param

This commit mimic the behaviour of the ignore_global_filters currently used with lucene syntax:
if you enable the ignore_global_filter option on data or on annotations, data or annotations
are filtered out when using the filter bar and the search bar

* Disable showQueryBar
@markov00 markov00 added the backport This PR is a backport of another PR label Feb 1, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@markov00 markov00 merged commit deccf5d into elastic:6.x Feb 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants