-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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(hdfs): Fix the failed unit tests in velox_hdfs_file_test #11884
Conversation
✅ Deploy Preview for meta-velox canceled.
|
20fb698
to
5f2b957
Compare
@majetideepak It seems that this issue is intermittent. I can't reproduce it locally, and it doesn't occur consistently on Velox CI either. |
@JkSelf It does look nondeterministic. The test seems to be passing now. |
5f2b957
to
b8bed20
Compare
@majetideepak Yes. I will close this pr. |
52cf355
to
c352bb2
Compare
c352bb2
to
6765a61
Compare
@majetideepak I have not yet identified the root cause. I suspect it may be due to the port being in use. I attempted to debug in the PR, but I was unable to reproduce the issue there. Do you have any suggestions |
6765a61
to
1f6bd74
Compare
@JkSelf I opened a PR here to fix the port issue https://github.com/facebookincubator/velox/pull/11996/files |
Fix #11857.