feat: emr instance fleet priority allocation #35731
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 2 commits
October 14, 2025 11:34
… strategy - Add PRIORITIZED option to OnDemandAllocationStrategy enum - Add optional priority field to InstanceTypeConfigProperty interface - Update JSON conversion to include priority in CloudFormation output - Add comprehensive unit and integration tests Enables users to specify instance type priorities for EMR instance fleets when using the prioritized allocation strategy, aligning CDK with AWS EMR API.
github-actions
bot
added
beginning-contributor
Daniel-ZA
changed the title
aws-cdk-automation
requested changes
Oct 13, 2025
There was a problem hiding this comment.
The pull request linter fails with the following errors:
❌ Features must contain a change to an integration test file and the resulting snapshot.
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.
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue # (if applicable)
Closes #35710
Reason for this change
The CDK's OnDemandAllocationStrategy enum only includes
LOWEST_PRICE, but AWS EMR supports a prioritized allocation strategy that allows users to specify instance type priorities. This enhancement aligns the CDK with the full EMR API capabilities.Description of changes
PRIORITIZED = 'prioritized'to theOnDemandAllocationStrategyenumpriority?: numberfield toInstanceTypeConfigPropertyDescribe any new or updated permissions being added
No new IAM permissions are required. This change only extends existing EMR cluster creation functionality with additional configuration options.
Description of how you validated changes
• Unit tests: Added 3 tests covering enum values, priority field functionality, and CloudFormation output validation
• Integration test: Created end-to-end test with realistic EMR cluster configuration using prioritized allocation strategy
• Manual validation: Verified CloudFormation templates generate correctly with AllocationStrategy: "prioritized" and Priority: N fields
• Backward compatibility: Confirmed existing tests continue to pass without modification
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license