-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
HDFS-16466. Implement Linux permission flags on Windows #4526
Conversation
* statinfo.cc uses POSIX permission flags. These flags aren't available for Windows. * This PR implements the equivalent flags on Windows to make this cross platform compatible.
* Since _WIN32 applies to both 32 and 64 bit versions of Windows.
This reverts commit da0cec1.
This comment was marked as outdated.
This comment was marked as outdated.
hadoop-hdfs-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/lib/x-platform/stat.h
Outdated
Show resolved
Hide resolved
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
* HDFS-16466. Implement Linux permission flags on Windows * statinfo.cc uses POSIX permission flags. These flags aren't available for Windows. * This PR implements the equivalent flags on Windows to make this cross platform compatible.
Description of PR
statinfo.cc uses POSIX permission flags. These flags aren't available for Windows. We need to implement the equivalent flags on Windows to make this cross platform compatible.
How was this patch tested?
Hadoop Jenkins CI validation.
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?