Skip to content

Commit

Permalink
[Auto Suggest] Fix Grammar Changes (opensearch-project#9120)
Browse files Browse the repository at this point in the history
* sql more field name tests

Signed-off-by: Paul Sebastian <[email protected]>

* undo search command change w/ updated logic, generalize rerun flag to work with any lang and any number of rules

Signed-off-by: Paul Sebastian <[email protected]>

* beginning of updating combined results properly

Signed-off-by: Paul Sebastian <[email protected]>

* Changeset file for PR opensearch-project#9120 created/updated

* cleaner non-recursive system to go through rerunWithout rule list and a general nonspecific combine function

Signed-off-by: Paul Sebastian <[email protected]>

* revert qualified name grammar

Signed-off-by: Paul Sebastian <[email protected]>

* revert from and table source grammars

Signed-off-by: Paul Sebastian <[email protected]>

* fix ppl field name issue

Signed-off-by: Paul Sebastian <[email protected]>

* remove ppl agg func snippets and update ppl field name test

Signed-off-by: Paul Sebastian <[email protected]>

* leave a space for sql agg func

Signed-off-by: Paul Sebastian <[email protected]>

* ppl autocomplete more pref. rules test

Signed-off-by: Paul Sebastian <[email protected]>

* remove backticks from ppl field name to match for value completion

Signed-off-by: Paul Sebastian <[email protected]>

* add comment explaining result combination

Signed-off-by: Paul Sebastian <[email protected]>

---------

Signed-off-by: Paul Sebastian <[email protected]>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Signed-off-by: Ubuntu <[email protected]>
  • Loading branch information
2 people authored and Ubuntu committed Jan 30, 2025
1 parent ead724b commit 53271d8
Show file tree
Hide file tree
Showing 15 changed files with 2,125 additions and 1,368 deletions.
2 changes: 2 additions & 0 deletions changelogs/fragments/9120.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
fix:
- PPL Grammar Parsing related issues ([#9120](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/9120))

Large diffs are not rendered by default.

Loading

0 comments on commit 53271d8

Please sign in to comment.