chore(glue-alpha): refactor spark jobs code-related props handling

[humanzz]

Issue # (if applicable)

Related to #33356

Reason for this change

address the feedback section of extraJars, extraFiles, extraPythonFiles, extraJarsFirst + refactor to reduce duplicative code across the different spark job classes

Description of changes

• update the different spark jobs props to
  • make extraPythonFiles, extraFiles, extraJars, and extraJarsFirst available in pyspark jobs
  • make extraFiles, extraJars, and extraJarsFirst available in scala spark jobs
• introduce a base SparkJob class and SparkJobProps to reduce duplicate logic across different spark job classes
  • introduce setupSparkCodeArguments protected method along with an interface SparkExtraCodeProps to handle setting up Code-related arguments
  • fix spark ui setup to be consistent with docs for sparkUI prop
  • merge content of spark-ui-utils.ts into spark-job.ts and retire it
  • cleanup comma expression for role setup
• fix typos and do renames to follow conventions

Describe any new or updated permissions being added

N/A

Description of how you validated changes

updated unit tests

Checklist

• My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

---

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.37%. Comparing base (7fe033a) to head (a155bb7).

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #33567   +/-   ##
=======================================
  Coverage   82.37%   82.37%           
=======================================
  Files         120      120           
  Lines        6933     6933           
  Branches     1169     1169           
=======================================
  Hits         5711     5711           
  Misses       1119     1119           
  Partials      103      103           

Flag	Coverage Δ
suite.unit	82.37% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
packages/aws-cdk	∅ <ø> (∅)
packages/aws-cdk-lib/core	82.37% <ø> (ø)

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[humanzz]

After the second commit, I had to export SparkExtraCodeProps and SparkUIConfiguration to pass JSII issues from the 1st commit.

Another thought I'm having is that there's a lot of duplicative logic in the different spark job classes, and I wonder if it can be simplified by either (1) Extracting common protected methods into Job or (2) introduction of a new intermediate base class for spark jobs, any thoughts?

[humanzz]

@GavinZZ this is a PR where I address my comments in #33238 (comment)

[GavinZZ]

Wow, this is awesome! Thank you for the refactoring and typo fixing!

[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).

[aws-cdk-automation]

AWS CodeBuild CI Report

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

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

[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).

[github-actions]

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.