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

Fix Python functions state which is completely broken #4027

Merged
merged 1 commit into from
Apr 11, 2019

Conversation

ivankelly
Copy link
Contributor

When a python functions instance is started with state enabled, it
crashes because the state client cannot connect to the state
endpoints. This is because the state client is never given the state
endpoints, so there's no way it ever could connect.

When a python functions instance is started with state enabled, it
crashes because the state client cannot connect to the state
endpoints. This is because the state client is *never* given the state
endpoints, so there's no way it ever could connect.
@ivankelly ivankelly self-assigned this Apr 11, 2019
@ivankelly ivankelly requested a review from merlimat April 11, 2019 14:19
@ivankelly
Copy link
Contributor Author

@aahmed-se (couldn't add as a reviewer, but please take a look)

@merlimat merlimat added this to the 2.3.2 milestone Apr 11, 2019
@merlimat merlimat added the type/bug The PR fixed a bug or issue reported a bug label Apr 11, 2019
@srkukarni
Copy link
Contributor

rerun java8 tests
rerun cpp tests

@merlimat merlimat merged commit b7108e8 into apache:master Apr 11, 2019
@jerrypeng
Copy link
Contributor

@ivankelly can we write a integration test for this?

merlimat pushed a commit that referenced this pull request May 19, 2019
When a python functions instance is started with state enabled, it
crashes because the state client cannot connect to the state
endpoints. This is because the state client is *never* given the state
endpoints, so there's no way it ever could connect.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants