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

Closes #67. Change the way DataNode host names are specified to be more resilient to hostname resolution. #68

Merged
merged 1 commit into from
Oct 10, 2018

Conversation

xkrogen
Copy link
Collaborator

@xkrogen xkrogen commented Oct 8, 2018

No description provided.

@xkrogen
Copy link
Collaborator Author

xkrogen commented Oct 8, 2018

Can't reproduce this locally, so submitting a patch to get a Travis CI run...

@xkrogen xkrogen force-pushed the ekrogen-build-fix branch from 7fc646c to 77c9d06 Compare October 8, 2018 19:05
@xkrogen
Copy link
Collaborator Author

xkrogen commented Oct 8, 2018

Travis CI passes again, and I tested this on our internal cluster as well. It looks like it fixes the issue.

I'm not totally sure what happened, but I suspect something changed in the Travis CI infrastructure such that the host name resolved differently, in a way that made the previous version no longer work properly.

@xkrogen xkrogen changed the title Closes #67. Attempt to fix the build by changing the way the host name is specified. Closes #67. Change the way DataNode host names are specified to be more resilient to hostname resolution. Oct 8, 2018
@xkrogen xkrogen merged commit 8c617c2 into linkedin:master Oct 10, 2018
@xkrogen xkrogen deleted the ekrogen-build-fix branch October 10, 2018 15:32
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