Skip to content

Conversation

@nargokul
Copy link
Contributor

@nargokul nargokul commented Jan 6, 2026

The ProcessingJob resource doesn't accept tags parameter during initialization. Remove tags from the transformed dict before passing to ProcessingJob constructor to prevent errors.

The tags are passed on to the actual create_processing_job call in the submit() function .

https://tiny.amazon.com/1h1h2lk2x

The ProcessingJob resource doesn't accept tags parameter during
initialization. Remove tags from the transformed dict before
passing to ProcessingJob constructor to prevent errors.
Copy link
Collaborator

@aviruthen aviruthen left a comment

Choose a reason for hiding this comment

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

I'm good with making this change. I just want to note that the ProcessingJob object won't have a tags attribute even though the user can set it via the SDK and can view their tag on console. Perhaps this is fine since I'm not sure what the use case is for customers retrieving tags from a ProcessingJob python object directly, but just wanted to point that out here!

@jam-jee
Copy link
Collaborator

jam-jee commented Jan 6, 2026

I'm good with making this change. I just want to note that the ProcessingJob object won't have a tags attribute even though the user can set it via the SDK and can view their tag on console. Perhaps this is fine since I'm not sure what the use case is for customers retrieving tags from a ProcessingJob python object directly, but just wanted to point that out here!

Good point. May be we need to check how to convey this message to user, so that they are aware that tags are not really passed to downstream processing.

@nargokul nargokul merged commit 4722723 into aws:master Jan 6, 2026
10 of 18 checks passed
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.

3 participants