Skip to content

included job queuing feature - #59

Merged
toryas merged 2 commits into
toryas:masterfrom
juanmavalenciap:feature/job-queuing
Feb 13, 2025
Merged

included job queuing feature#59
toryas merged 2 commits into
toryas:masterfrom
juanmavalenciap:feature/job-queuing

Conversation

@juanmavalenciap

Copy link
Copy Markdown
Contributor

This change will allow the plugin to use the recently added job queuing feature:
https://aws.amazon.com/blogs/big-data/introducing-job-queuing-to-scale-your-aws-glue-workloads/

@monil1334 monil1334 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LG, can we have that it similar to MaxConcurrentRuns and rename it to JobRunQueuingEnabled @toryas

@juanmavalenciap

Copy link
Copy Markdown
Contributor Author

@monil1334 would you like me to make any additional changes to this PR?
I just wanted to kindly point out that MaxConcurrentRuns and JobRunQueuingEnabled are two distinct attributes defined in the documentation
https://docs.aws.amazon.com/glue/latest/dg/aws-glue-api-jobs-job.html

Comment thread src/domain/glue-job.ts Outdated

@monil1334 monil1334 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LG, @toryas can you please merge this feature request?

@david-roberts-payrix

Copy link
Copy Markdown

Hello, I wanted to see when we think that this change may be merged and released? Thanks.

@toryas
toryas merged commit 2c13c2c into toryas:master Feb 13, 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.

4 participants