feat(glue-alpha): add cfn-glue-table-tableinput-parameters to Glue table construct

[lawofcycles]

Add cfn-glue-table-tableinput-parameters to Glue Table construct as optional props

User can specify additional table parameter when creating Glue Table.
Any key/value can be set depending on each user's requirement like table's additional metadata or statistics. Some parameter can be used when AWS services / 3rd party tools read table like skip.header.line.count.

Closes #14159.

---

All Submissions:

• Have you followed the guidelines in our Contributing guide?
  Adding new Unconventional Dependencies:
• This PR adds new unconventional dependencies following the process described here
  New Features
• Have you added the new feature to an integration test?
  • Did you use yarn integ to deploy the infrastructure and generate the snapshot (i.e. yarn integ without --dry-run)?

---

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

[lawofcycles]

Note: On current glue-alpha,skip.header.line.count is set on wrong parameter section, cfn-glue-table-storagedescriptor-parameters on table class.
It should be set to cfn-glue-table-tableinput-parameters because other services like Athena expect it. This issue is reported on #27365 and I may fix it on another pull request in the future.

[lawofcycles]

It seems integration test failed on CodeBuild. I'll check Build Logs.

Lerna (powered by Nx) Running targets build, test for 63 projects failed

Tasks not run because their dependencies failed or --nx-bail=true:

• @aws-cdk/aws-apigatewayv2-authorizers-alpha:test
• @aws-cdk/aws-iot-actions-alpha:test
• @aws-cdk/aws-iotevents-actions-alpha:test
• @aws-cdk/aws-scheduler-targets-alpha:test

Failed tasks:

• @aws-cdk/aws-glue-alpha:test

real 18m14.153s
user 209m8.220s
sys 13m29.698s

[lawofcycles]

I have resolved some issues. I would appreciate it if you could review it.

[vinayak-kukreja]

Hey @lawofcycles , apologies for the delay and thank you for opening a contribution with us. I will review this PR soon.

[vinayak-kukreja]

Awaiting @TheRealAmazonKendra review on the PR.

[bcolas]

Any chance to have it reviewed quickly : this is a issue that avoid us to push S3Table in CI/CD to production : no access to change CloudFormation template in production environment

[kaizencc]

FYI: we're in the process of revamping the glue module, which is why I haven't been approving any glue PRs lately. Please bear with us a bit longer, I'm trying to see if we can get the green light to continue to merge quality-of-life PRs in the meantime.

[natalie-white-aws]

Per Kendra's request, I reviewed these changes and can confirm that these changes do not conflict with the glue-alpha L2 work in flight. Safe to merge pending all other CDK service team checks without impacting our work.

[mergify]

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[mergify]

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[sumupitchayan]

@Mergifyio update

[mergify]

update

❌ Mergify doesn't have permission to update

For security reasons, Mergify can't update this pull request. Try updating locally.
GitHub response: refusing to allow a GitHub App to create or update workflow .github/workflows/close-stale-prs.yml without workflows permission

Pull request has been modified.

[aws-cdk-automation]

AWS CodeBuild CI Report

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

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