-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
ARROW-10003: [C++] Create parent dir for any destination fs in CopyFiles #8187
Conversation
4d0703c
to
4ad9abd
Compare
@pitrou CI failure seems unrelated, PTAL |
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 one potential concern.
} | ||
|
||
std::vector<std::string> dirs(destination_dirs.size()); | ||
std::move(destination_dirs.begin(), destination_dirs.end(), dirs.begin()); |
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.
Interesting, I didn't know this std::move
override!
…systems in CopyFiles
1430d19
to
6a8bd29
Compare
Non-Apache Travis: https://travis-ci.org/github/bkietz/arrow/builds/730077798 |
Somehow #8187 had a passing build but it didn't regenerate the R docs, so R builds are failing on master because the doc signature doesn't match the code. Closes #8277 from nealrichardson/fix-r Authored-by: Neal Richardson <neal.p.richardson@gmail.com> Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
Previously only the parent directories of files to be copied to a different filesystem were created. Closes apache#8187 from bkietz/10003-Create-directories-in-Cop Authored-by: Benjamin Kietzman <bengilgit@gmail.com> Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
Somehow apache#8187 had a passing build but it didn't regenerate the R docs, so R builds are failing on master because the doc signature doesn't match the code. Closes apache#8277 from nealrichardson/fix-r Authored-by: Neal Richardson <neal.p.richardson@gmail.com> Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
Previously only the parent directories of files to be copied to a different filesystem were created. Closes apache#8187 from bkietz/10003-Create-directories-in-Cop Authored-by: Benjamin Kietzman <bengilgit@gmail.com> Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
Somehow apache#8187 had a passing build but it didn't regenerate the R docs, so R builds are failing on master because the doc signature doesn't match the code. Closes apache#8277 from nealrichardson/fix-r Authored-by: Neal Richardson <neal.p.richardson@gmail.com> Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
Previously only the parent directories of files to be copied to a different filesystem were created.