-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Support index pattern selector syntax in ES|QL #120660
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
Changes from 8 commits
1c151f5
9e70be8
3fb8d9d
8856a0d
4be6984
4b92237
e2297ed
278800c
87d0897
9fd861e
28e5825
418e12a
e39ca3a
738eef0
12fdb9b
9b635f3
aea1d89
60a8011
d69ea04
208cea6
8df5cc2
b3edb8d
16831d7
c11fbc6
eca0b0c
c88f1f8
7d582ba
577e15d
c0bc7d8
f13b674
c101724
2373537
67a7ccd
c4b0a93
6d5b049
364bbd4
7f8680d
80088a0
29ed978
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -139,12 +139,17 @@ fromCommand | |
|
||
indexPattern | ||
: (clusterString COLON)? indexString | ||
| {this.isDevVersion()}? (clusterString COLON)? indexString (CAST_OP selectorString)? | ||
|
||
; | ||
|
||
clusterString | ||
: UNQUOTED_SOURCE | ||
; | ||
|
||
selectorString | ||
: UNQUOTED_SOURCE | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we allow quoting selector? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I tried to simply follow how the definition for the clusterString are laid out. I don't have too much of an opinion on quoting the selector or not. The selector contents can only ever be |
||
; | ||
|
||
indexString | ||
: UNQUOTED_SOURCE | ||
| QUOTED_STRING | ||
|
Large diffs are not rendered by default.
Uh oh!
There was an error while loading. Please reload this page.