Skip to content

Conversation

evanchooly
Copy link
Member

Just what it says on the tin.

Ultimately, I'd like to replace that bootstrap script with an actual native binary but we'll need quarkus to support command line apps first.

use a standard, default mapping since the actual path is used only by the extension, typically
@dmlloyd
Copy link
Member

dmlloyd commented Apr 3, 2019

I wonder if we really need command line app support to make this work fully within a single process. Perhaps there's some combination of reactive REST client, plus CDI startup listener or application scope bean, which would allow an initial request to be fired with a completion stage that takes the response, feeds it through the REST machinery to the REST handler, takes the response from that, and feeds it back to Lambda as a response request.

@evanchooly
Copy link
Member Author

I tried the CDI listener approach for a while and had issues with it. However, having figured out all the other bits, i'm happy to take another swing at it. But in another PR. ;)

Copy link
Member

@dmlloyd dmlloyd left a comment

Choose a reason for hiding this comment

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

This will do for a start.

@evanchooly evanchooly merged commit c8ba30f into quarkusio:master Apr 3, 2019
@evanchooly evanchooly deleted the lambda branch April 5, 2019 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants