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

[Watcher UI] Inconsistent query syntax validation between Watcher UI and Console #18425

Closed
elasticmachine opened this issue Aug 28, 2017 · 5 comments
Labels
Feature:Console Dev Tools Console Feature Feature:Watcher Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more

Comments

@elasticmachine
Copy link
Contributor

Original comment by @ppf2:

6.0 Alpha 2

The following is considered valid syntax (and works) if you create the watch using the Console (but it is considered invalid syntax by Watcher UI):

"body": """{"query": {"bool": {"must": {"query_string": {"query": "test"}}}}}"""

Ideally, it will be nice for both editors to support 3 ticks for quotes.

The following is considered valid syntax in both Console and Watcher, but when you actually try to put the watch, it fails.

"body": {"query": {"bool": {"must": {"query_string": {"query": "test"}}}}}

The following works in both:

"body": "{\"query\": {\"query_string\": {\"query\": \"test\"}}}"
@elasticmachine
Copy link
Contributor Author

Original comment by @ppf2:

@bohyun-e It will be nice to use consistent parsers for Watcher UI and Console. I am having to rewrite watches yet another time for customers because they want to use the Watcher UI, but it's failing syntax left and right.

@elasticmachine
Copy link
Contributor Author

Original comment by @bohyun-e:

@ppf2 thanks for bringing this up.

cc: @chrisronline @BigFunger as well as @bmcconaghy (welcome! 🎆 )

@elasticmachine
Copy link
Contributor Author

Original comment by @Rasroh:

cc @elastic/kibana-management

@cjcenizal cjcenizal added Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more and removed :Management DO NOT USE labels May 1, 2019
@elasticmachine
Copy link
Contributor Author

Pinging @elastic/es-ui

@alisonelizabeth
Copy link
Contributor

Duplicate of #21752 and resolved by #57563.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Console Dev Tools Console Feature Feature:Watcher Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more
Projects
None yet
Development

No branches or pull requests

3 participants