Skip to content

Commit

Permalink
Update cpp/src/arrow/filesystem/s3fs.cc
Browse files Browse the repository at this point in the history
Co-authored-by: Antoine Pitrou <pitrou@free.fr>
  • Loading branch information
jorisvandenbossche and pitrou authored Dec 5, 2023
1 parent 1d887b7 commit ffda582
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions cpp/src/arrow/filesystem/s3fs.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2410,9 +2410,9 @@ class S3FileSystem::Impl : public std::enable_shared_from_this<S3FileSystem::Imp
DCHECK_GT(file_info.path().size(), bucket.size());
auto file_path = file_info.path().substr(bucket.size() + 1);
if (file_info.IsDirectory()) {
// the selector returns FileInfo objects for directories with a
// a path that never ends in a trailing slash, but for S3 the file
// needs to have a trailing slash to recognize it as direcotory
// The selector returns FileInfo objects for directories with a
// a path that never ends in a trailing slash, but for AWS the file
// needs to have a trailing slash to recognize it as directory
// (https://github.com/apache/arrow/issues/38618)
DCHECK_OK(internal::AssertNoTrailingSlash(file_path));
file_path = file_path + kSep;
Expand Down

0 comments on commit ffda582

Please sign in to comment.