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

store handler return value in redis #178

Closed

Conversation

albertjan
Copy link

This makes it possible to return something from the handler. Thus giving the job poster the ability to wait for a job to complete and get the return value.

Sister PR on oxen: fsprojects-archive/zzarchive-oxen#38

@albertjan
Copy link
Author

@manast I've rebased this 😄

@manast
Copy link
Member

manast commented Sep 21, 2015

great stuff by the way ;)

@albertjan
Copy link
Author

Thanks. 😄 I will update the pr tomorrow.

@albertjan albertjan force-pushed the store-handler-return-value branch 2 times, most recently from d66c4f9 to 72d957e Compare September 22, 2015 08:26
Albert-Jan Nijburg added 2 commits September 22, 2015 10:26
and include storing it in the lua script in `moveToSet`
@TylerBrock
Copy link

What can I do to help get this merged? @albertjan can you restart the tests to make sure the failure wasn't just a fluke?

@rossholdway
Copy link

It would be great to get this merged!

@manast
Copy link
Member

manast commented Nov 22, 2015

@albertjan should I merge this?

@albertjan
Copy link
Author

I'll rebase first. Then if you think its good I think it could go in.

On Sunday, 22 November 2015, Manuel Astudillo [email protected]
wrote:

@albertjan https://github.com/albertjan should I merge this?


Reply to this email directly or view it on GitHub
#178 (comment).

@TylerBrock
Copy link

@albertjan any updates on the rebase?

@TylerBrock
Copy link

I've tried to fix the linter errors that the tests have but I can't seem to reproduce on my machine.

@albertjan
Copy link
Author

I'll try just started a new job :)

On Monday, 23 November 2015, Tyler Brock [email protected] wrote:

@albertjan https://github.com/albertjan any updates on the rebase?


Reply to this email directly or view it on GitHub
#178 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants