-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[dagster-aws] [docs] add docs for PipesEMRClient #25011
base: master
Are you sure you want to change the base?
[dagster-aws] [docs] add docs for PipesEMRClient #25011
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @danielgafni and the rest of your teammates on Graphite |
803e1ef
to
09de78c
Compare
3331660
to
2e6ae16
Compare
09de78c
to
2135d63
Compare
2e6ae16
to
d9eb9cf
Compare
2135d63
to
e15f1e1
Compare
d9eb9cf
to
dc793c9
Compare
e15f1e1
to
d425b47
Compare
dc793c9
to
5231c8f
Compare
d425b47
to
4eb3226
Compare
5231c8f
to
79d63ab
Compare
4eb3226
to
cee41a0
Compare
79d63ab
to
e26cc22
Compare
cee41a0
to
f810d74
Compare
e26cc22
to
fe2f8b6
Compare
f810d74
to
a461aa3
Compare
fe2f8b6
to
4e9beda
Compare
a461aa3
to
5f70c47
Compare
4e9beda
to
e9a2b0e
Compare
5f70c47
to
c158f37
Compare
e9a2b0e
to
3297448
Compare
c158f37
to
d2501ef
Compare
3297448
to
633284d
Compare
d2501ef
to
0b5d016
Compare
633284d
to
2c1d9a9
Compare
0b5d016
to
34f526d
Compare
2c1d9a9
to
56ba07e
Compare
e7bda0b
to
2e05446
Compare
40c2508
to
96a5f2c
Compare
96a5f2c
to
77c836b
Compare
@@ -93,7 +93,7 @@ def run( | |||
Args: | |||
context (OpExecutionContext): The context of the currently executing Dagster op or asset. | |||
params (dict): Parameters for the ``start_job_run`` boto3 AWS EMR Serverless client call. | |||
See `Boto3 API Documentation <https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/emr-serverless/client/start_job_run.html>`_ | |||
See `Boto3 EMR Serverless API Documentation <https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/emr-serverless/client/start_job_run.html>`_ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sphinx was complaining about this entry being identical to another one so I decided to inject service name here
@@ -122,6 +122,10 @@ Clients | |||
|
|||
.. autoclass:: dagster_aws.pipes.PipesECSClient | |||
|
|||
.. autoclass:: dagster_aws.pipes.PipesEMRClient | |||
|
|||
.. autoclass:: dagster_aws.pipes.PipesEMRServerlessClient |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops... was missing
Deploy preview for dagster-docs-beta ready! Preview available at https://dagster-docs-beta-j9nl5efgy-elementl.vercel.app Direct link to changed pages: |
77c836b
to
614c8da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall this looks great--just a couple of small comments.
614c8da
to
bd54eec
Compare
Merge activity
|
bd54eec
to
5c1b2f9
Compare
Summary & Motivation
Adding API docs & tutorial for AWS EMR Pipes
How I Tested These Changes