hadoop-yetus commented on a change in pull request #533: HADOOP-14630 Contract
Tests to verify create, mkdirs and rename under a file is forbidden
URL: https://github.com/apache/hadoop/pull/533#discussion_r264008750
##########
File path:
hadoop-common-project/hadoop-common/src/site/markdown/filesystem/filesystem.md
##########
@@ -524,9 +524,17 @@ Create a directory and all its parents
#### Preconditions
+The path must either be a directory or not exist
+
if exists(FS, p) and not isDir(FS, p) :
raise [ParentNotDirectoryException, FileAlreadyExistsException,
IOException]
+No ancestor may be a file
+
+ forall d = ancestors(FS, p) :
+ if exists(FS, d) and not isDir(FS, d) :
+ raise [ParentNotDirectoryException, FileAlreadyExistsException,
IOException]
+
Review comment:
whitespace:end of line
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
