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
Workaround JDK bug 8034057 #12970
Workaround JDK bug 8034057 #12970
Conversation
/** | ||
* Files.getFileStore(Path) useless here! Don't complain, just try it yourself. | ||
*/ | ||
@SuppressForbidden(reason = "works around the bugs") |
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.
just for kicks can we have reason = "works around https://bugs.openjdk.java.net/browse/JDK-8034057"
it's just more obvious which bug is meant
left some minor comments LGTM otherwise |
LGTM too |
LGTM |
This is pretty ugly code, but the bug is even uglier, ES won't start at all, plugin installation won't work, etc. So I don't know what versions we want this in, someone other than me decide. FYI I also sent an email to nio-dev, asking about the real fix... |
I am ok with this going into 2.0 |
Fixed in e07f039 |
JDK backport link: https://bugs.openjdk.java.net/browse/JDK-8133891 |
See https://bugs.openjdk.java.net/browse/JDK-8034057, where two filesystem methods are broken for SUBST'd drive letters (this is basically where you map a local drive to another letter, like c:\foo -> d:).
These bugs won't be fixed until java 9, but its a configuration some people use on windows.