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

End-to-End Async Query API PPL tests #1036

Merged
merged 2 commits into from
Feb 7, 2025

Conversation

normanj-bitquill
Copy link
Contributor

Description

Adding Async Query PPL tests.

Related Issues

#1022

Check List

  • Updated documentation (docs/ppl-lang/README.md)
  • Implemented unit tests
  • Implemented tests for combination with other commands
  • New added source code should include a copyright header
  • Commits are signed per the DCO using --signoff

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.

@normanj-bitquill
Copy link
Contributor Author

This PR also includes changes so that the mys3.default prefix can be used on the spark and spark-worker containers.

Copy link
Member

Choose a reason for hiding this comment

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

@normanj-bitquill can you please add javaDoc on the class & each method pupose ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have added Javadoc comments.

Copy link
Member

@YANG-DB YANG-DB left a comment

Choose a reason for hiding this comment

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

Looks good - just make sure the existing E2E documentation are up to date with these changes

@normanj-bitquill
Copy link
Contributor Author

Looks good - just make sure the existing E2E documentation are up to date with these changes

@YANG-DB The documentation looks up to date. I took a look over it.

This PR is based on PR #1028 so I will need to rebase after #1028 is merged.

@normanj-bitquill
Copy link
Contributor Author

@YANG-DB This PR has been rebased off of main.

@YANG-DB
Copy link
Member

YANG-DB commented Feb 7, 2025

@normanj-bitquill can you please check the failed CI ?

* Some test are currently disabled due to #3298

Signed-off-by: Norman Jordan <[email protected]>
@normanj-bitquill
Copy link
Contributor Author

@YANG-DB I rebased with the latest change in main. The CI ran successfully.

@penghuo penghuo merged commit 1a6b9f3 into opensearch-project:main Feb 7, 2025
4 checks passed
@normanj-bitquill normanj-bitquill deleted the e2e-async-ppl-tests branch February 11, 2025 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants