steveloughran commented on a change in pull request #2046:
URL: https://github.com/apache/hadoop/pull/2046#discussion_r453894382
##########
File path:
hadoop-common-project/hadoop-common/src/site/markdown/filesystem/fsdatainputstreambuilder.md
##########
@@ -68,6 +68,11 @@ This is relevant with those stores which return version/etag
information,
including the S3A and ABFS connectors -they MAY use this to guarantee that
the file they opened is exactly the one returned in the listing.
+There is no requirement for `status.getPath()` to the resolved path of
Review comment:
The reason for this is that viewfs and the hive fs wrappers have their
own schemas/paths. If we insist on the paths matching, you woudn't be able to
use the API through hive or mounted viewfs, which would be too brittle. Saying
"it's just a hint" avoids this issue. Same reason I'm not worrying about type
of the status.
s3a FS: use etag and version ID, if known
other status: use the length
----------------------------------------------------------------
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]
