Skip to content

Method to enable functions worker in pulsar container #2711

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

Merged
merged 6 commits into from
Jun 30, 2020

Conversation

lanwen
Copy link
Contributor

@lanwen lanwen commented May 11, 2020

This change should simplify a bit pulsar functions development experience
Tried here: https://github.com/lanwen/pulsar-functions-example

Without waiter pulsar could return random 503, depending on how fast
was a call.

@lanwen lanwen requested review from bsideup, kiview and rnorth as code owners May 11, 2020 22:36
lanwen added 3 commits May 14, 2020 15:39
This change should simplify a bit pulsar functions development experience
Tried here: https://github.com/lanwen/pulsar-functions-example

Without waiter pulsar could return random 503, depending on how fast
was a call.
@lanwen lanwen force-pushed the pulsar_functions branch from 2ba29f9 to 45fddfd Compare May 14, 2020 13:39
@lanwen lanwen requested a review from bsideup May 14, 2020 16:15
@lanwen
Copy link
Contributor Author

lanwen commented Jun 30, 2020

@bsideup is anything I can do in this PR?

@bsideup bsideup added this to the next milestone Jun 30, 2020
@bsideup
Copy link
Member

bsideup commented Jun 30, 2020

@lanwen thanks for the ping! Just applied a minor change, will merge once CI is green 👍

@bsideup bsideup merged commit 6186598 into testcontainers:master Jun 30, 2020
@lanwen lanwen deleted the pulsar_functions branch October 28, 2020 18:14
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