-
Notifications
You must be signed in to change notification settings - Fork 4k
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
fix(stepfunctions): support JSONATA for result writer and item reader #33404
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #33404 +/- ##
=======================================
Coverage 80.92% 80.92%
=======================================
Files 236 236
Lines 14255 14255
Branches 2491 2491
=======================================
Hits 11536 11536
Misses 2434 2434
Partials 285 285
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Marking this PR as Also update the PR label to correctly reflect the issue priority. |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Issue # (if applicable)
Closes #33396.
Reason for this change
Result writer and item reader class do not support using JSONATA. Deployment will fails due to if SFN is set to use JSONATA, it expects
Arguments
in the ASL instead ofParameters
.Description of changes
Support
JSONATA
as the query language.Description of how you validated changes
New unit test and new integ test with assertions
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license