Skip to content

Conversation

@alsenydiallo
Copy link
Collaborator

Issue #, if available:

Description of changes:

  • Update Dockerfile to use Python 3.12 Lambda base image
  • Convert broken multi-stage build to single-stage build
  • Update dependency versions: Hadoop 3.3.6, AWS SDK 1.12.261, PySpark 3.5.0
  • Switch from yum to dnf package manager
  • Add support for requirements.txt if present
  • Update spark-class script to prioritize Java 11 Amazon Corretto paths
  • Fix aws-ecr-repository-push.sh script argument validation and ECR login
  • Add proper error handling and usage instructions
  • Update all Python paths from 3.10 to 3.12 throughout
  • Add diagnostics for spark-class installation and verification

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

- Update Dockerfile to use Python 3.12 Lambda base image
- Convert broken multi-stage build to single-stage build
- Update dependency versions: Hadoop 3.3.6, AWS SDK 1.12.261, PySpark 3.5.0
- Switch from yum to dnf package manager
- Add support for requirements.txt if present
- Update spark-class script to prioritize Java 11 Amazon Corretto paths
- Fix aws-ecr-repository-push.sh script argument validation and ECR login
- Add proper error handling and usage instructions
- Update all Python paths from 3.10 to 3.12 throughout
- Add diagnostics for spark-class installation and verification
@alsenydiallo alsenydiallo requested a review from JohnChe88 July 31, 2025 15:09
@JohnChe88 JohnChe88 self-assigned this Jul 31, 2025
Copy link
Member

@JohnChe88 JohnChe88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, latest version of Python 3.12 base image and 3.5.1 spark version update will make it more secure

@JohnChe88 JohnChe88 merged commit fbff227 into main Jul 31, 2025
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.

2 participants