Skip to content
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

Feature Request: Add support for ResultSelector #95

Open
brightsparc opened this issue Oct 9, 2020 · 4 comments
Open

Feature Request: Add support for ResultSelector #95

brightsparc opened this issue Oct 9, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@brightsparc
Copy link
Contributor

ResultPath is supported in the SDK, but the newer ResultSelector is not which is preferrable for Lambda functions which emit a bunch of metadata in addition to the JSON output result.

See docs

@a13zen
Copy link

a13zen commented Jul 2, 2021

Anny chance this could be pushed through? Seems like the linked PR for this is almost ready?

@wong-a wong-a added the enhancement New feature or request label Jul 9, 2021
@wong-a
Copy link
Contributor

wong-a commented Jul 9, 2021

@a13zen We're taking another look at the PR to get this shipped soon. Sorry for the delay!

@lasdem
Copy link

lasdem commented Aug 9, 2022

Unfortunately I see that last activity here was a year ago, any chance this is getting implemented?

@lasdem
Copy link

lasdem commented Aug 9, 2022

For anyone else coming here looking for a solution until this is implemented, I found a workaround that works for me.
I put a Pass step after my lambda step and use the parameters field of the pass step the same way as you would use the ResultSelector on the output of my lambda step.

lambda_run_step = LambdaStep(
    state_id="lambda_run_step",
    parameters={
        "FunctionName": "db_run_procedure", 
        "Payload": {
            "name": "helloworld", 
            "schema": "dummy",
            "database": "example"
        }
    }
)

lambda_run_output_step = Pass(
    state_id="lambda_run_output_step ",
    parameters = {
        "lambda_run_result.$": "$.Payload"
    }
)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
4 participants