-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
base: trunk
Are you sure you want to change the base?
Conversation
💔 -1 overall
This message was automatically generated. |
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?
while IFS= read -r -d '' file | ||
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. |
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-binary
files?