Skip to content

ES|QL: Check if cluster aliases and index patterns are valid before executing query #122497

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.

Sign up for GitHub

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

Merged
merged 28 commits into from
Jun 25, 2025
Merged

ES|QL: Check if cluster aliases and index patterns are valid before executing query #122497

Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
8f324ae
Check if index patterns conform to valid format before validation
pawankartik-elastic Feb 13, 2025
1ddbce4
Update docs/changelog/122497.yaml
pawankartik-elastic Feb 13, 2025
46758cb
Let `validateClusterString()` look for `REMOTE_CLUSTER_INDEX_SEPARATOR`
pawankartik-elastic Feb 13, 2025
2f2ee85
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Feb 13, 2025
7ca6d70
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Apr 7, 2025
9dd96df
Mute 3 cases till further clarification and fix error string message
pawankartik-elastic Apr 7, 2025
0686c8f
Fix tests
pawankartik-elastic Apr 7, 2025
7c55d6c
Fix bug in breaking down indices
pawankartik-elastic Apr 8, 2025
2653377
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Apr 10, 2025
5832ff0
Tiny refactoring around how wildcard is processed and added tests
pawankartik-elastic Apr 10, 2025
6cdbf5a
[CI] Auto commit changes from spotless
elasticsearchmachine Apr 10, 2025
c2160bb
Drop duplicated test cases and fix flaky-ness caused by quoting
pawankartik-elastic Apr 10, 2025
544171b
Set cluster string to `null` when it cannot be associated with an index
pawankartik-elastic Apr 11, 2025
133a7c7
Generate correct invalid patterns
pawankartik-elastic Apr 11, 2025
7da29d8
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Apr 11, 2025
5c70d51
Address review comments and don't break indices into its constituents
pawankartik-elastic Apr 16, 2025
7ad6ac3
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Jun 18, 2025
3058610
Adhere to the new grammar
pawankartik-elastic Jun 19, 2025
376126e
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Jun 19, 2025
1a43029
Update docs/changelog/122497.yaml
pawankartik-elastic Jun 19, 2025
8363b70
Update x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/…
pawankartik-elastic Jun 20, 2025
9c16afb
Apply suggestions from review
pawankartik-elastic Jun 23, 2025
8f9f519
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Jun 23, 2025
b6339e7
Apply suggestions from code review
pawankartik-elastic Jun 23, 2025
0cc5580
Apply suggestions from review
pawankartik-elastic Jun 23, 2025
816fcc1
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Jun 23, 2025
0346762
Do not mention asterisk as an invalid char
pawankartik-elastic Jun 25, 2025
9d275a5
Merge branch 'main' into pkar/index-pattern-check
pawankartik-elastic Jun 25, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into pkar/index-pattern-check
  • Loading branch information
@pawankartik-elastic
pawankartik-elastic authored Apr 11, 2025
commit 7da29d8e2aa80557c1102168e686768647969a0f

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.