-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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 db_stress for custom env #5122
Conversation
Summary: unnecessary because HdfsLogger is not the owner of writable file. The owner will close it on close or destruction.
Different hadoop distributions may have different ways to configure. It would be nice if somebody can have a script setting up environment variables, classpath, etc. for different hadoop distributions. I'm considering removing the config-related part of this PR, and commit only the code changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@riversand963 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@riversand963 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@riversand963 merged this pull request in d77476e. |
Summary: Fix some hdfs-related code so that it can compile and run 'db_stress' Pull Request resolved: facebook#5122 Differential Revision: D14675495 Pulled By: riversand963 fbshipit-source-id: cac280479efcf5451982558947eac1732e8bc45a
Fix some hdfs-related code so that it can compile and run 'db_stress'
Test plan:
The stack trace printed at the beginning of the test results from HDFS client which catches the
RemoteException
and decides to print the stack trace.