dongjoon-hyun commented on a change in pull request #28940:
URL: https://github.com/apache/spark/pull/28940#discussion_r448109490
##########
File path:
common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/ExecutorDiskUtils.java
##########
@@ -50,14 +58,18 @@ public static File getFile(String[] localDirs, int
subDirsPerLocalDir, String fi
* the internal code in java.io.File would normalize it later, creating a
new "foo/bar"
* String copy. Unfortunately, we cannot just reuse the normalization code
that java.io.File
* uses, since it is in the package-private class java.io.FileSystem.
+ *
+ * On Windows, separator "\" is used instead of "/".
+ *
+ * "\\" is legal character in path name on Unix like OS, but illegal on
Windows.
Review comment:
Maybe,
- `is legal character` -> `is a legal character`.
- `Unix like` -> `Unix-like`.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
