-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HDFS-16930. Update the wrapper for fuse-dfs #5449
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
Conversation
|
💔 -1 overall
This message was automatically generated. |
jojochuang
left a comment
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.
Thanks for the PR. Has this been tested on different machines, such as x86 Linux or Mac?
| do | ||
| export CLASSPATH=$CLASSPATH:$file | ||
| done < <(find "$HADOOP_HOME/hadoop-hdfs-project" -name "*.jar" -print0) | ||
| done < <(find "$HADOOP_HOME/hadoop-tools/hadoop-distcp" -name "*.jar" -print0) |
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.
It's been a while but I don't think fuse-dfs relies on distcp jars
|
If some of these things are not clear then we can maybe split out the bits that can definitely be fixed. I've stumbled across this as well tonight. |
|
We're closing this stale PR because it has been open for 100 days with no activity. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. |
Description of PR
update path with LIBHDFS_PATH
add path to LD_LIBRARY_PATH with compatible to java11
edit CLASSPATH
How was this patch tested?
manual test
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?