Skip to content

Commit

Permalink
RFC for Glue L2 Construct (#498)
Browse files Browse the repository at this point in the history
* RFC for Glue L2 Construct

* Fix markdown linter findings

* Fix the rest of the markdown linter findings

* Updates after initial review - script files, job patterns, workflow and trigger definitions

* Forgot to run the linter again

* Do not use Deprecated Python versions

* Added parameter table, incorporating other review changes for Glue L2

* Fix table renderig for Glue L2

* remove extra chars, fix table alignment

* Removed action verbs, added role to each example, included ts code syntax highlighting, updated first section title.

* Add comprehensive parameter interfaces to each job type per RFC review

* Add mandatory workflow language for Notify Event Triggers

* Added worker type to parameter lists, updated timeout Duration type, added note about why IAM Role is required from the developer rather than being auto-generated by the L2

* Addressing @humanzz comments on Glue L2 extrajars parameters

* Added README section back to doc, changed from 3rd to first person, changed passive verbs to active, added more README style details, added runtime override for Ray jobs

---------

Co-authored-by: Janardhan Molumuri <janmuri@amazon.com>
Co-authored-by: Kendra Neil <53584728+TheRealAmazonKendra@users.noreply.github.com>
  • Loading branch information
3 people authored Aug 29, 2023
1 parent 4382064 commit c2883c7
Showing 1 changed file with 1,328 additions and 0 deletions.
Loading

0 comments on commit c2883c7

Please sign in to comment.