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

Remove unimplemented syntax #439

Merged
merged 4 commits into from
Jul 23, 2024

Conversation

ykmr1224
Copy link
Collaborator

@ykmr1224 ykmr1224 commented Jul 20, 2024

Description

  • Remove unimplemented syntax so that frontend can utilize the .g4 files for validation/auto-complete.
  • I considered comment-out, but ended up deleting them as we can refer this PR or file in SQL plugin to see the original implementation, and it became a bit messy when I commented out them with TODO tag.

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@ykmr1224 ykmr1224 force-pushed the unavailable-commands branch 8 times, most recently from 23efc45 to 3b4a57e Compare July 23, 2024 04:17
@ykmr1224 ykmr1224 changed the title Comment out not implemented commands Remove unimplemented syntax Jul 23, 2024
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
@ykmr1224 ykmr1224 force-pushed the unavailable-commands branch from 3b4a57e to a0f1fd7 Compare July 23, 2024 04:29
@ykmr1224 ykmr1224 marked this pull request as ready for review July 23, 2024 15:46
Copy link
Collaborator

@penghuo penghuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

ykmr1224 added 3 commits July 23, 2024 12:30
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Copy link
Collaborator

@noCharger noCharger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the change! Curious how to identify unimplemented ones? maybe add some test?

@YANG-DB YANG-DB merged commit 3e964be into opensearch-project:main Jul 23, 2024
4 checks passed
@ykmr1224
Copy link
Collaborator Author

Thanks for the change! Curious how to identify unimplemented ones? maybe add some test?

Some commands had implementation which throws error, so I referred them first. I found some more unimplemented ones, and I verified those syntax fails by running it in test.

@ykmr1224 ykmr1224 deleted the unavailable-commands branch July 23, 2024 23:06
@dai-chen dai-chen added bug Something isn't working 0.5 backport 0.4 labels Jul 26, 2024
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 29, 2024
* Remove unimplemented syntax

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Fix keywordsCanBeId and statsFunctionName

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Remove describe and show datasources

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Fix indent

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

---------

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
(cherry picked from commit 3e964be)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
dai-chen pushed a commit that referenced this pull request Jul 29, 2024
* Remove unimplemented syntax



* Fix keywordsCanBeId and statsFunctionName



* Remove describe and show datasources



* Fix indent



---------


(cherry picked from commit 3e964be)

Signed-off-by: Tomoyuki Morita <moritato@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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.5 backport 0.4 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants