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

Added Rollups and Transforms APIs #797

Merged
merged 2 commits into from
Jun 13, 2024
Merged

Conversation

nhtruong
Copy link
Collaborator

Description

Added Rollups and Transforms APIs via a generator.
Right now the generator cannot generate the types of these functions, yet. It's a much more complicated task that's in the work. The typing system in this repo also needs an overhaul.

Issues Resolved

closes #744

Check List

  • New functionality includes testing.
    • All tests pass
  • Linter check was successfull - yarn run lint doesn't show any errors
  • Commits are signed per the DCO using --signoff
  • Changelog was updated.

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.

@dblock
Copy link
Member

dblock commented Jun 12, 2024

Add some tests or at least a working sample?

@nhtruong
Copy link
Collaborator Author

nhtruong commented Jun 12, 2024

Going to add samples for both namespaces. For now, please double check if the generated code is correct.

@dblock
Copy link
Member

dblock commented Jun 13, 2024

Signed-off-by: Theo Truong <theotr@amazon.com>
#
Signed-off-by: Theo Truong <theotr@amazon.com>
@nhtruong nhtruong merged commit ac5ed87 into opensearch-project:main Jun 13, 2024
63 of 64 checks passed
@nhtruong nhtruong deleted the rollups branch June 13, 2024 16:30
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.

[FEATURE] plans to support rollup & transform APIs?
2 participants