-
Notifications
You must be signed in to change notification settings - Fork 867
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
[Autocomplete] PPL Autocomplete #7810
[Autocomplete] PPL Autocomplete #7810
Conversation
Signed-off-by: Eric <menwe@amazon.com>
Signed-off-by: Eric <menwe@amazon.com>
❌ Empty Changelog SectionThe Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section. |
Signed-off-by: Eric <menwe@amazon.com>
Signed-off-by: Eric <menwe@amazon.com>
Signed-off-by: Eric <menwe@amazon.com>
Signed-off-by: Eric <menwe@amazon.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feature/autocomplete-2.0 #7810 +/- ##
============================================================
- Coverage 63.83% 60.49% -3.35%
============================================================
Files 3661 3657 -4
Lines 81345 86844 +5499
Branches 12978 13324 +346
============================================================
+ Hits 51926 52532 +606
- Misses 26234 31143 +4909
+ Partials 3185 3169 -16
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Eric <menwe@amazon.com>
Signed-off-by: Eric <menwe@amazon.com>
Signed-off-by: Eric <menwe@amazon.com>
@@ -85,7 +85,8 @@ | |||
"cypress:run-with-security": "env TZ=America/Los_Angeles NO_COLOR=1 cypress run --headless --env SECURITY_ENABLED=true,openSearchUrl=https://localhost:9200,WAIT_FOR_LOADER_BUFFER_MS=500", | |||
"osd:ciGroup10": "echo \"dashboard_sanity_test_spec.js\"", | |||
"osd:ciGroup11": "echo \"apps/vis_builder/*.js\"", | |||
"generate:opensearchsqlantlr": "./node_modules/antlr4ng-cli/index.js -Dlanguage=TypeScript -o ./src/plugins/data/public/antlr/opensearch_sql/.generated -visitor -no-listener -Xexact-output-dir ./src/plugins/data/public/antlr/opensearch_sql/grammar/OpenSearchSQLLexer.g4 ./src/plugins/data/public/antlr/opensearch_sql/grammar/OpenSearchSQLParser.g4" | |||
"generate:opensearchsqlantlr": "./node_modules/antlr4ng-cli/index.js -Dlanguage=TypeScript -o ./src/plugins/data/public/antlr/opensearch_sql/.generated -visitor -no-listener -Xexact-output-dir ./src/plugins/data/public/antlr/opensearch_sql/grammar/OpenSearchSQLLexer.g4 ./src/plugins/data/public/antlr/opensearch_sql/grammar/OpenSearchSQLParser.g4", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice. i see what you did. another backlog item could be moving the scripts to a file to the scripts folder and then call the script like yarn generate:antler --sql
import { QuerySuggestion } from '../../autocomplete'; | ||
import { IndexPattern, IndexPatternField } from '../../index_patterns'; | ||
|
||
const fetchFieldSuggestions = ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think there can be a space in the language manager we where we register code completion stuff so that these files can be put in the query enhancements plugin.
cc: @ashwin-pc, @abbyhu2000 what do you think?
Merging into feature branch. |
3ae20ef
into
opensearch-project:feature/autocomplete-2.0
* add initial ppl autocomplete Signed-off-by: Eric <menwe@amazon.com> * untrack .antlr files Signed-off-by: Eric <menwe@amazon.com> * ignore intermediate grammar auto-gen files Signed-off-by: Eric <menwe@amazon.com> * add rules and related functionalities Signed-off-by: Eric <menwe@amazon.com> * Changeset file for PR opensearch-project#7810 created/updated * Changeset file for PR opensearch-project#7810 created/updated * minor comment cleanning Signed-off-by: Eric <menwe@amazon.com> * add ppl generation command Signed-off-by: Eric <menwe@amazon.com> * add rules Signed-off-by: Eric <menwe@amazon.com> * correct typo Signed-off-by: Eric <menwe@amazon.com> * fix inserting text issue Signed-off-by: Eric <menwe@amazon.com> * remove colon for PPL field Signed-off-by: Eric <menwe@amazon.com> --------- Signed-off-by: Eric <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
* add initial ppl autocomplete Signed-off-by: Eric <menwe@amazon.com> * untrack .antlr files Signed-off-by: Eric <menwe@amazon.com> * ignore intermediate grammar auto-gen files Signed-off-by: Eric <menwe@amazon.com> * add rules and related functionalities Signed-off-by: Eric <menwe@amazon.com> * Changeset file for PR opensearch-project#7810 created/updated * Changeset file for PR opensearch-project#7810 created/updated * minor comment cleanning Signed-off-by: Eric <menwe@amazon.com> * add ppl generation command Signed-off-by: Eric <menwe@amazon.com> * add rules Signed-off-by: Eric <menwe@amazon.com> * correct typo Signed-off-by: Eric <menwe@amazon.com> * fix inserting text issue Signed-off-by: Eric <menwe@amazon.com> * remove colon for PPL field Signed-off-by: Eric <menwe@amazon.com> --------- Signed-off-by: Eric <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
* add initial ppl autocomplete Signed-off-by: Eric <menwe@amazon.com> * untrack .antlr files Signed-off-by: Eric <menwe@amazon.com> * ignore intermediate grammar auto-gen files Signed-off-by: Eric <menwe@amazon.com> * add rules and related functionalities Signed-off-by: Eric <menwe@amazon.com> * Changeset file for PR opensearch-project#7810 created/updated * Changeset file for PR opensearch-project#7810 created/updated * minor comment cleanning Signed-off-by: Eric <menwe@amazon.com> * add ppl generation command Signed-off-by: Eric <menwe@amazon.com> * add rules Signed-off-by: Eric <menwe@amazon.com> * correct typo Signed-off-by: Eric <menwe@amazon.com> * fix inserting text issue Signed-off-by: Eric <menwe@amazon.com> * remove colon for PPL field Signed-off-by: Eric <menwe@amazon.com> --------- Signed-off-by: Eric <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
* [Autocomplete - SQL] Minor interface change to add suggestion type and move suggestion provider registration location (#7758) * add table/source as prefix to suggested fields Signed-off-by: Eric <menwe@amazon.com> * add type to column Signed-off-by: Eric <menwe@amazon.com> * move registeration to osd/monaco Signed-off-by: Eric <menwe@amazon.com> * add detail Signed-off-by: Eric <menwe@amazon.com> * Changeset file for PR #7758 created/updated --------- Signed-off-by: Eric <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> * [Autocomplete] PPL Autocomplete (#7810) * add initial ppl autocomplete Signed-off-by: Eric <menwe@amazon.com> * untrack .antlr files Signed-off-by: Eric <menwe@amazon.com> * ignore intermediate grammar auto-gen files Signed-off-by: Eric <menwe@amazon.com> * add rules and related functionalities Signed-off-by: Eric <menwe@amazon.com> * Changeset file for PR #7810 created/updated * Changeset file for PR #7810 created/updated * minor comment cleanning Signed-off-by: Eric <menwe@amazon.com> * add ppl generation command Signed-off-by: Eric <menwe@amazon.com> * add rules Signed-off-by: Eric <menwe@amazon.com> * correct typo Signed-off-by: Eric <menwe@amazon.com> * fix inserting text issue Signed-off-by: Eric <menwe@amazon.com> * remove colon for PPL field Signed-off-by: Eric <menwe@amazon.com> --------- Signed-off-by: Eric <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> * small interface updates Signed-off-by: Paul Sebastian <paulstn@amazon.com> * small type update Signed-off-by: Paul Sebastian <paulstn@amazon.com> * make inclusion for pipe, comma, and equal tokens Signed-off-by: Paul Sebastian <paulstn@amazon.com> * refactor and generalize field fetcher for dql and ppl, including details Signed-off-by: Paul Sebastian <paulstn@amazon.com> * use field fetching util for sql and update sugg detail for dql Signed-off-by: Paul Sebastian <paulstn@amazon.com> * detail for ppl Signed-off-by: Paul Sebastian <paulstn@amazon.com> * create range parameter to help identify suggestions with whitespace, implemented dql value WS suggs Signed-off-by: Paul Sebastian <paulstn@amazon.com> * single line editor overflow initial override for sugg window Signed-off-by: Paul Sebastian <paulstn@amazon.com> * update dql tests to account for details and value ranges Signed-off-by: Paul Sebastian <paulstn@amazon.com> --------- Signed-off-by: Eric <menwe@amazon.com> Signed-off-by: Paul Sebastian <paulstn@amazon.com> Co-authored-by: Eric Wei <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
* [Autocomplete - SQL] Minor interface change to add suggestion type and move suggestion provider registration location (#7758) * add table/source as prefix to suggested fields Signed-off-by: Eric <menwe@amazon.com> * add type to column Signed-off-by: Eric <menwe@amazon.com> * move registeration to osd/monaco Signed-off-by: Eric <menwe@amazon.com> * add detail Signed-off-by: Eric <menwe@amazon.com> * Changeset file for PR #7758 created/updated --------- Signed-off-by: Eric <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> * [Autocomplete] PPL Autocomplete (#7810) * add initial ppl autocomplete Signed-off-by: Eric <menwe@amazon.com> * untrack .antlr files Signed-off-by: Eric <menwe@amazon.com> * ignore intermediate grammar auto-gen files Signed-off-by: Eric <menwe@amazon.com> * add rules and related functionalities Signed-off-by: Eric <menwe@amazon.com> * Changeset file for PR #7810 created/updated * Changeset file for PR #7810 created/updated * minor comment cleanning Signed-off-by: Eric <menwe@amazon.com> * add ppl generation command Signed-off-by: Eric <menwe@amazon.com> * add rules Signed-off-by: Eric <menwe@amazon.com> * correct typo Signed-off-by: Eric <menwe@amazon.com> * fix inserting text issue Signed-off-by: Eric <menwe@amazon.com> * remove colon for PPL field Signed-off-by: Eric <menwe@amazon.com> --------- Signed-off-by: Eric <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> * small interface updates Signed-off-by: Paul Sebastian <paulstn@amazon.com> * small type update Signed-off-by: Paul Sebastian <paulstn@amazon.com> * make inclusion for pipe, comma, and equal tokens Signed-off-by: Paul Sebastian <paulstn@amazon.com> * refactor and generalize field fetcher for dql and ppl, including details Signed-off-by: Paul Sebastian <paulstn@amazon.com> * use field fetching util for sql and update sugg detail for dql Signed-off-by: Paul Sebastian <paulstn@amazon.com> * detail for ppl Signed-off-by: Paul Sebastian <paulstn@amazon.com> * create range parameter to help identify suggestions with whitespace, implemented dql value WS suggs Signed-off-by: Paul Sebastian <paulstn@amazon.com> * single line editor overflow initial override for sugg window Signed-off-by: Paul Sebastian <paulstn@amazon.com> * update dql tests to account for details and value ranges Signed-off-by: Paul Sebastian <paulstn@amazon.com> --------- Signed-off-by: Eric <menwe@amazon.com> Signed-off-by: Paul Sebastian <paulstn@amazon.com> Co-authored-by: Eric Wei <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit 0245540) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* [Autocomplete - SQL] Minor interface change to add suggestion type and move suggestion provider registration location (#7758) * add table/source as prefix to suggested fields * add type to column * move registeration to osd/monaco * add detail * Changeset file for PR #7758 created/updated --------- * [Autocomplete] PPL Autocomplete (#7810) * add initial ppl autocomplete * untrack .antlr files * ignore intermediate grammar auto-gen files * add rules and related functionalities * Changeset file for PR #7810 created/updated * Changeset file for PR #7810 created/updated * minor comment cleanning * add ppl generation command * add rules * correct typo * fix inserting text issue * remove colon for PPL field --------- * small interface updates * small type update * make inclusion for pipe, comma, and equal tokens * refactor and generalize field fetcher for dql and ppl, including details * use field fetching util for sql and update sugg detail for dql * detail for ppl * create range parameter to help identify suggestions with whitespace, implemented dql value WS suggs * single line editor overflow initial override for sugg window * update dql tests to account for details and value ranges --------- (cherry picked from commit 0245540) Signed-off-by: Eric <menwe@amazon.com> Signed-off-by: Paul Sebastian <paulstn@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Eric Wei <menwe@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Description
Issues Resolved
Screenshot
Testing the changes
Changelog
Check List
yarn test:jest
yarn test:jest_integration