feat(stepfunction-tasks): allow string in outputLocation parameter

[razin99]

Issue currently faced

I tried to do the following:

new AthenaStartQueryExecution(
  stack,
  'Start Athena Query',
  {
    queryString: sfn.JsonPath.stringAt('$.queryString'),
    queryExecutionContext: {
      databaseName: 'database',
    },
    resultConfiguration: {
      encryptionConfiguration: {
        encryptionOption: EncryptionOption.S3_MANAGED,
      },
     outputLocation: {
        bucketName: 'query-results-bucket',
        objectKey: sfn.JsonPath.stringAt('$.folder'),
    },
  });

But was unable to due to an error message:

Field references must be the entire string, cannot concatenate them (found 's3://$[Token]/$[Token]')

Solution

Allow outputLocation to receive string. Then the new parameter can also be the following:

outputLocation: sfn.JsonPath.format(
  's3://query-results-bucket/folder/{}',
  sfn.JsonPath.format('$.folder'),
)

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[aws-cdk-automation]

The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.

A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
• Commit ID: 89c2541
• Result: SUCCEEDED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[mrgrain]

Unfortunately using a union type here will make any other language but typescript very unusable. They just turn into an any equivalent.

[mrgrain]

I see you have encountered an issue and I appreciate your drive to just go an fix it! In this case however, would you mind open an issue first? This gives our triage and support team a chance to have a look at it, possibly come with a workaround and at least inform the solution better.

[aws-cdk-automation]

This PR has been in the CHANGES REQUESTED state for 3 weeks, and looks abandoned. To keep this PR from being closed, please continue work on it. If not, it will automatically be closed in a week.

[aws-cdk-automation]

This PR has been deemed to be abandoned, and will be automatically closed. Please create a new PR for these changes if you think this decision has been made in error.