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

prevent cleanUpNode fn from duplicating jobs resolves #454 #523

Merged
merged 1 commit into from
Sep 5, 2017

Conversation

jsnoble
Copy link
Member

@jsnoble jsnoble commented Sep 5, 2017

this will also add slicer_hostname and slicer_port to the execution context

@godber
Copy link
Member

godber commented Sep 5, 2017

The integration tests pass and there is no currently known reliable way to test the multi-slicer issue so I am going to accept this.

@godber godber merged commit e4d94b8 into terascope:master Sep 5, 2017
@jsnoble jsnoble deleted the job_duplication branch November 21, 2017 14:36
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.

2 participants